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

should suggest pushing inventory variables down to role if higher scope is uneccesary #18

Open
aioue opened this issue Sep 17, 2015 · 1 comment

Comments

@aioue
Copy link

aioue commented Sep 17, 2015

atk-toolkit should find variables that are only used in a single role variables, and suggest that they are moved down into that role variables directory as they do not need a larger scope.

@dellis23
Copy link
Owner

I'm not sure how this would be detected. How do you know if a role that is using the variable is only applied to a single host? Unfortunately, given the dynamic nature of every component used by Ansible (python, jinja, yaml), I don't know that the room for static analysis like this will make sense. But I'm very interested in hearing what you envisioned.

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

No branches or pull requests

3 participants