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

Detect unused hiera variables #56

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

mcanevet
Copy link
Member

@mcanevet mcanevet commented Jun 1, 2016

No description provided.

@JoshuaSmeda
Copy link

Would really like this feature!

@mcanevet
Copy link
Member Author

I rebased it, but TBH I can't remember whether or not it was working when I coded this 6 years ago... As I don't use Puppet anymore, I can't test this. I guess someone has to test it to see if if works properly and we could merge it.

@JoshuaSmeda
Copy link

JoshuaSmeda commented Aug 17, 2022

Thanks for merging in!

I tested it a couple of weeks ago, but it doesn't work correctly yet. When I was in this space, I got blocked on this issue:

https://github.com/voxpupuli/puppet-ghostbuster/pull/56/files#diff-b35760b16358995b4318148cd42c05ab94665ec9f003d09717e8dfcd161ee179R30

Whereas, the new version of Puppet has a different format for keys, as depicted here: https://puppet.com/docs/puppet/6/hiera_config_yaml_5.html

There might be more issues, but only invested about 5 minutes of free time I had and found that issue so long.

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

Successfully merging this pull request may close these issues.

2 participants