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

Fix variable name in config_context_{list, _hash} #59

Closed
wants to merge 1 commit into from
Closed

Fix variable name in config_context_{list, _hash} #59

wants to merge 1 commit into from

Conversation

ayosec
Copy link

@ayosec ayosec commented Jun 13, 2018

Fix a typo in the variable name used in the code for both config_context_list and config_context_hash.

Without the fix, when I try to use these methods in strict mode, I get the following error:

Mixlib::Config::UnknownConfigOptionError:
 Reading unsupported config value symbol.

The included tests can reproduce the error if you revert the changes in lib/mixlib/config.rb

@ayosec ayosec requested a review from a team June 13, 2018 03:26
@thommay
Copy link
Contributor

thommay commented Jun 13, 2018

Duplicate of #57

@thommay thommay marked this as a duplicate of #57 Jun 13, 2018
@thommay
Copy link
Contributor

thommay commented Jun 13, 2018

Thanks for the PR. Given that we've got two identical ones I'm going to take the earlier one. Sorry for the delay in getting this merged.

@thommay thommay closed this Jun 13, 2018
@thommay
Copy link
Contributor

thommay commented Jun 13, 2018

The fix is released in 2.2.8

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