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

(MAINT) Ensure parameters are canonicalized #75

Merged
merged 2 commits into from Dec 9, 2020
Merged

(MAINT) Ensure parameters are canonicalized #75

merged 2 commits into from Dec 9, 2020

Conversation

michaeltlombardi
Copy link
Contributor

Parameters are never returned from the get method,
which prevented them from being included in the
canonicalized resource hash passed to the set method.

This commit re-adds them to the canonicalized hash.

Parameters are never returned from the get method,
which prevented them from being included in the
canonicalized resource hash passed to the set method.

This commit re-adds them to the canonicalized hash.
@michaeltlombardi michaeltlombardi requested a review from a team as a code owner December 9, 2020 21:13
@michaeltlombardi michaeltlombardi added this to In progress in DSC via automation Dec 9, 2020
DSC automation moved this from In progress to Reviewer approved Dec 9, 2020
@RandomNoun7 RandomNoun7 merged commit d7e3426 into puppetlabs:main Dec 9, 2020
DSC automation moved this from Reviewer approved to Done Dec 9, 2020
@michaeltlombardi michaeltlombardi deleted the maint/main/fix-parameter-canonicalization branch December 9, 2020 21:24
pmcmaw added a commit to pmcmaw/ruby-pwsh that referenced this pull request Jan 19, 2021
pmcmaw added a commit to pmcmaw/ruby-pwsh that referenced this pull request Jan 19, 2021
adrianiurca added a commit that referenced this pull request Jan 20, 2021
(GH-75) Including module name in vendored module path
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
DSC
Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants