Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adopt automation good practices #26

Open
ericzolf opened this issue Dec 18, 2021 · 1 comment
Open

Adopt automation good practices #26

ericzolf opened this issue Dec 18, 2021 · 1 comment

Comments

@ericzolf
Copy link

As Ansible-practitioner, I must admit that the current code makes my eyes bleed from time to time (sorry for the tone, really just joking).

I would suggest to adopt some "better" practices, also for more consistency in the code. As basis I'd recommend https://github.com/redhat-cop/automation-good-practices which is maintained by the practitioners from Red Hat. That's anyway what I'm following for my contributions.

It doesn't mean that the existing code needs to be changed at once, just that we have a common basis to discuss about quality of PRs, i.e. future code. Anyway, I wanted to start the discussion.

@seahawk1986
Copy link
Member

As a hobbyist with a single larger Ansible project that is about four years older than the best practice guide I have to admit that I never heard of this document - especially the early Ansible documentation was also quite vague at times on how to do things - this feels a little like C++ where you can do a lot of nasty stuff supported by the standard and several years later there is a nice guide by Stroustrup, Sutter et al, that tells you how to do things with a subset of the language instead after there is tons of old code ...

That being said if it helps to improve readability and maintainability I am open to improvements.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants