Comments (3)
The trailing newline linting error in YAML files threw me for a loop when I started using molecule as well. My objection is the same, namely that it feels distinctly anti-POSIX, but having a style pragma enforced by tooling is arguably better than bickering about preferences.
To put myself in the maintainers' shoes, adding an option to permit trailing newlines, but continue linting on all other attributes, would open the door for a nightmare of configurability. You can, after all, excuse directories from the linting, which may be the best course of action for you if you don't want to trim your YAML files. For spec and other files, doesn't seem like we have much recourse at this time.
from molecule.
Molecule's validator is not checking for trailing newlines (that the last line matches /\n$/
), it is checking for trailing blank lines, that is that the last line of the file should not be empty (matches /^\n$/
). This is an important distinction.
https://github.com/metacloud/molecule/blob/master/molecule/validators.py#L96
from molecule.
it is checking for trailing blank lines
Thank you for clearing this up. Great explanation, and I now agree with the implementation.
from molecule.
Related Issues (20)
- Update documentation about default Driver HOT 1
- [Docs] Update user guide with missing sections HOT 4
- [Docs] Add porting guide HOT 1
- dependency step installs roles to user's roles path, not in ephemeral directory HOT 3
- CRITICAL 'molecule/default/molecule.yml' glob failed. Exiting - Unable to run molecule HOT 13
- Unknown command: ansible-playbook
- Switch Releases to CalVer HOT 1
- molecule matrix command produces unreadable colored output HOT 1
- Container drivers are not working
- Is `v24.2.0` wrongly tagged? HOT 1
- wants tmpfs to be dict, not list HOT 8
- Molecule config file in {project}/.config/molecule/config.yml not found HOT 1
- Molecule instance inherits all groups from inventory
- [Docs] Add driver name for podman HOT 2
- The env option of the shell dependency has no effect
- Unable to Create Temporary Directory During Molecule Test Execution in Ansible Role HOT 2
- Molecule converge with specific scenario ends up with an error HOT 1
- The molecule with docker driver doesn't work with requests lib => 32.x.x HOT 1
- Project cleanup
- Getting 404 errors from github while trying to install roles HOT 4
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from molecule.