-
Notifications
You must be signed in to change notification settings - Fork 582
Remove dig and dig44 functions #1316
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
Conversation
Looks like some of the unit tests for Other functions used Dig? ill have to figure out how to unhook all of that *** fixed*** 2 layers of functional deprecation in this one... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm surprised that the REFERENCE.md wasn't updated. The CI probably doesn't check for it (we recently implemented. I don't think you need to regenerate it for each PR, but during the release PR we need to ensure it's up2date (puppetlabs_spec_helper has a rake task for that I think). CC: @chelnak
I think vox pupuli calls this: release_checks -> validate -> https://github.com/puppetlabs/puppetlabs_spec_helper/blob/main/lib/puppetlabs_spec_helper/rake_tasks.rb#L213
I think if this is the same as my workflows on my team, REFERENCE.MD is updated as part of the release prep workflow |
Yeah I believe that is correct |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
dig became part of puppet core in Puppet 4, and even the compatibility function dig44 does not work in puppet 6 or later
this should be removed.
As part of this PR try_get_value was also removed as it was deprecated in favour of dig 6 years ago