Enforce ansible-lint - Githubissues

roots / trellis

WordPress LEMP stack with PHP 8.2, Composer, WP-CLI and more

https://roots.io/trellis/

MIT License

2.51k stars 607 forks source link

Enforce ansible-lint #43

Closed austinpray closed 10 years ago

austinpray commented 10 years ago

Hey guys,

Seeing as though @swalkinshaw and I are sticklers for things like formatting, I figured we should also be sticklers for ansible best-practices.

Perhaps we should put https://github.com/willthames/ansible-lint on our radar.

[ ] clean up existing warnings
[ ] monitor pull requests for further warnings

austinpray commented 10 years ago

Example:

Austins-MacBook-Pro:bedrock-ansible austinpray$ ansible-lint site.yml 
[ANSIBLE0006] cp used in place of copy module
/Users/austinpray/Dropbox/DEV/opensauce/bedrock-ansible/roles/nginx/tasks/main.yml:0
Task/Handler: Copy over h5bp configuration

austinpray commented 10 years ago

Nevermind this linter is dumb

nathanielks commented 10 years ago

lol. @austinpray I've enjoyed digging through the closed issues and seeing how many you've opened then subsequently closed yourself =p

austinpray commented 10 years ago

It looks like my issue with this linter might turn into a feature! https://github.com/willthames/ansible-lint/issues/21

nathanielks commented 10 years ago

ha! Look at you!