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

Honor default separator in set_config #545

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

kajinamit
Copy link
Contributor

... to use the consistent separator by default in all functions by default. Users can still override the separator by passing the separator argument when calling the function.

Fixes: #544

Summary

Provide a detailed description of all the changes present in this pull request.

Additional Context

Add any additional context about the problem here.

  • Root cause and the steps to reproduce. (If applicable)
  • Thought process behind the implementation.

Related Issues (if any)

#544

Checklist

  • 🟢 Spec tests.
  • 🟢 Acceptance tests.
  • Manually verified. (For example puppet apply)

@kajinamit kajinamit requested review from bastelfreak, ekohl, smortex and a team as code owners September 20, 2024 12:26
... to use the consistent separator by default in all functions by
default. Users can still override the separator by passing
the separator argument when calling the function.

Fixes: puppetlabs#544
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.

Puppet::Util::Inifile: set_value does not honor the default key_val_separator
1 participant