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

value field in ini_setting is now always a string #164

Merged
merged 1 commit into from
Jan 22, 2016
Merged

value field in ini_setting is now always a string #164

merged 1 commit into from
Jan 22, 2016

Conversation

bovy89
Copy link
Contributor

@bovy89 bovy89 commented Jan 14, 2016

https://forge.puppetlabs.com/puppetlabs/inifile

value
Optional. Supplies a value for the specified setting. Valid options: a string. Default value: undefined.

@jyaworski
Copy link
Member

Hello:

Does this misbehave on older versions of inifile?

@bovy89
Copy link
Contributor Author

bovy89 commented Jan 21, 2016

I tested it against version 1.0.3, 1.2.0 and 1.4.3 of puppetlabs-inifile module.
Value must be a string or an update of this value will not modify the file

@jyaworski
Copy link
Member

Good enough for me. Merging.

jyaworski added a commit that referenced this pull request Jan 22, 2016
value field in ini_setting is now always a string
@jyaworski jyaworski merged commit cc0952c into voxpupuli:master Jan 22, 2016
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