ci: fixed ansible-lint errors and warnings

geerlingguy / ansible-role-php-versions

Ansible Role - PHP Versions

https://galaxy.ansible.com/geerlingguy/php-versions/

MIT License

98 stars 73 forks source link

ci: fixed ansible-lint errors and warnings #61

Closed webarchitect609 closed 3 years ago

webarchitect609 commented 3 years ago

fixed: error "syntax-check 'src' is not a valid attribute for a Play";
fixed: error "internal-error the role 'geerlingguy.repo-remi' was not found in ...";
fixed: warning "[line-length] line too long (173 > 120 characters)".

geerlingguy commented 3 years ago

Thanks! I'm holding off on touching anything more in my roles currently until a Galaxy bug gets worked out, so I'll wait until then to consider merging this.

geerlingguy commented 3 years ago

Hmm... I might use this though (contrary to what I said in the issue you raised earlier...).

webarchitect609 commented 3 years ago

@geerlingguy , could you please consider using ansible-lint again? The Galaxy bug you were waiting for is now fixed. I've updated my PR and triple-checked everything, including the latest ansible-lint docs compliance. And finally, I've rebased & squashed everything in a single commit, so please, checkout my new PR #66