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

Fixes #34079 - Allow explicit strings in key=value options #356

Merged
merged 1 commit into from Dec 12, 2021

Conversation

ofedoren
Copy link
Member

@ofedoren ofedoren commented Dec 3, 2021

@adamruzicka, sorry for disturbing you with this, but since you reviewed the issue, could you please confirm this fixes the issue and doesn't bring new ones? :)

Copy link
Contributor

@adamruzicka adamruzicka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I played with it for a bit and I like it. I'd say this is the way it should be.

@ofedoren
Copy link
Member Author

ofedoren commented Dec 8, 2021

@adiabramovitch, could you please check this one as well? :) Also, please let me know if you don't have merge rights (you should have, but recently there was a cleaning up in the teams of the organization, so I'm not quite sure if it's still true)

@adiabramovitch
Copy link
Contributor

adiabramovitch commented Dec 12, 2021

@ofedoren Thanks Oleh!
Just notice to use name="\"*\""

@adiabramovitch adiabramovitch merged commit 9375f2d into theforeman:master Dec 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants