Ansible-lint and yamllint #32

Merged
Ghost merged 16 commits from linting into dev 2020-06-08 17:14:32 +02:00

16 commits

Author SHA1 Message Date
Patrick Marchand 9f378fc1f9 Misunderstood syntax for unordered lists in markdown
Some checks failed
continuous-integration/drone/push Build is failing
continuous-integration/drone/pr Build is failing
So an unordered list needs three spaces and an ordered list needs
a dot and two spaces.
2020-06-04 13:27:03 -04:00
Patrick Marchand b711154722 Apply fix in last commit to other markdown files
Some checks failed
continuous-integration/drone/push Build is failing
continuous-integration/drone/pr Build is failing
So a markdown list should be no indentation followed by two spaces. '  '

*  one
*  two
2020-06-04 13:23:06 -04:00
Patrick Marchand 87ff4e635f Try to fix markdown indentation after reading issues
Some checks failed
continuous-integration/drone/push Build is failing
continuous-integration/drone/pr Build is failing
See https://github.com/remarkjs/remark-lint/issues/178
2020-06-04 13:18:28 -04:00
Patrick Marchand ce09c3f81f Try to fix markdown indentation, again and again
Some checks failed
continuous-integration/drone/push Build is failing
continuous-integration/drone/pr Build is failing
2020-06-04 13:16:08 -04:00
Patrick Marchand 245c80d73f Try to fix markdown indentation, again
Some checks failed
continuous-integration/drone/pr Build is failing
continuous-integration/drone/push Build is failing
2020-06-04 13:14:23 -04:00
Patrick Marchand aaf22d450e Try to fix markdown indentation
Some checks failed
continuous-integration/drone/pr Build is failing
continuous-integration/drone/push Build is failing
2020-06-04 13:12:38 -04:00
Patrick Marchand db488ba8ef Split long lines in git role
Some checks failed
continuous-integration/drone/pr Build is failing
continuous-integration/drone/push Build is failing
2020-06-04 13:10:08 -04:00
Patrick Marchand c6410ca1f5 fix forgotten quote in .drone.yml
Some checks failed
continuous-integration/drone/push Build is failing
continuous-integration/drone/pr Build is failing
2020-06-04 13:02:47 -04:00
Patrick Marchand 98089a3274 Fix yaml lint lines too long
Some checks failed
continuous-integration/drone/push Build is failing
continuous-integration/drone/pr Build is failing
In some cases I used block scalars: https://yaml-multiline.info/
In other cases I added newlines
In rare cases I just ignored the rule: https://yamllint.readthedocs.io/en/stable/disable_with_comments.html
2020-06-04 12:51:53 -04:00
Patrick Marchand e877b721f9 Fix readme markdown
Some checks failed
continuous-integration/drone/push Build is failing
continuous-integration/drone/pr Build is failing
2020-06-04 11:47:08 -04:00
Patrick Marchand 896548c14c lint markdown files
Some checks failed
continuous-integration/drone/pr Build is failing
continuous-integration/drone/push Build is failing
Makes sure the documentation will render correctly
2020-06-04 11:42:46 -04:00
Patrick Marchand f8d6a0be86 add a yaml linting step to the ci
Some checks failed
continuous-integration/drone/push Build is failing
continuous-integration/drone/pr Build is failing
catches problems in the drone ci file as well
2020-06-04 11:41:49 -04:00
Patrick Marchand 4e8fca725c mention linting in CONTRIBUTING.md
Some checks failed
continuous-integration/drone/push Build is failing
continuous-integration/drone/pr Build is failing
more of a test for the drone ci hook honestly
2020-06-04 11:21:21 -04:00
Patrick Marchand a6213719ae Add linting step to the ci
We will need to go through code and ignore any warnings we dont want to fix.
2020-06-03 16:45:42 -04:00
Jérémy Dubois e29e0e9e62 Ansible-lint and yamllint again
Lot of truthy variables, indentation and trailing spaces
2020-06-01 11:37:15 +02:00
Patrick Marchand af7b3b36fe Ansible-lint and yamllint
Does not fix all warnings, but gets rid of the purely cosmetic ones.
(roles/accounts/tasks/main.yml)
2020-05-22 11:49:18 -04:00