diff options
author | Adrien Vergé <adrienverge@gmail.com> | 2016-04-18 17:25:39 +0200 |
---|---|---|
committer | Matt Clay <matt@mystile.com> | 2016-11-11 14:50:57 -0800 |
commit | 527d8307c133100a3ca585589d590c598608ef07 (patch) | |
tree | d3506f32dd9dd7a275a24ceab896c3f89b3afcad /.yamllint | |
parent | 0e834fc9e4ace93a5e47647fb3544d775977d90c (diff) | |
download | ansible-527d8307c133100a3ca585589d590c598608ef07.tar.gz |
Lint YAML files under test/
This commit extends YAML linting by enabling standard rules from the
`yamllint` tool [1]. Since syntax errors and key duplicates are already
checked since 4d48711, this change only adds detection for cosmetic
problems. It also narrows checks to the test/ dir only.
The main goal is to prevent future problems to enter the code base
without being noticed. While it would be a huge effort to be PEP8
compliant, it is relatively easy to have correct YAML style *now* and
prevent future errors by enabling linting.
Note: for those (like me) caring about code attribution: use `git blame
-w` to ignore whitespace-only changes.
Note: I disabled some linting checks (such as indentation), they can be
enforced in the future if needed. Similarly, current checks can also be
disabled. See the `.yamllint` file.
[1]: https://yamllint.readthedocs.io/
Diffstat (limited to '.yamllint')
-rw-r--r-- | .yamllint | 17 |
1 files changed, 7 insertions, 10 deletions
@@ -1,22 +1,19 @@ extends: default -# Disable all cosmetic rules -# (see https://github.com/ansible/ansible/pull/15470#issuecomment-214437876) -# Only keep 'key-duplicates' and 'new-lines: {type: unix}' checks enabled. - rules: - braces: disable - brackets: disable - colons: disable - commas: disable + braces: {max-spaces-inside: 1, level: error} + brackets: {max-spaces-inside: 1, level: error} + colons: {max-spaces-after: -1, level: error} + commas: {max-spaces-after: -1, level: error} comments: disable comments-indentation: disable document-start: disable - empty-lines: disable - hyphens: disable + empty-lines: {max: 3, level: error} + hyphens: {level: error} indentation: disable key-duplicates: enable line-length: disable new-line-at-end-of-file: disable new-lines: {type: unix} trailing-spaces: disable + truthy: disable |