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) Use strings instead of symbols for os names. #2149

Merged
merged 1 commit into from
Oct 21, 2020

Conversation

BogdanIrimie
Copy link
Contributor

@BogdanIrimie BogdanIrimie commented Oct 21, 2020

If symbols are used for os names, the matching is not done correctly and the os hierarchy cannot be detected.

@BogdanIrimie BogdanIrimie added the bug Something isn't working label Oct 21, 2020
@BogdanIrimie BogdanIrimie requested review from a team October 21, 2020 11:44
@BogdanIrimie
Copy link
Contributor Author

Jenkins please test this on all

@oanatmaria oanatmaria merged commit 4636328 into puppetlabs:main Oct 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants