-
Notifications
You must be signed in to change notification settings - Fork 23.7k
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
Migrate away from pipes.quote #56785
Conversation
This comment has been minimized.
This comment has been minimized.
@mkrizek Thanks for working on this. I'd split out the ansible-test changes into a separate PR so they can be backported independently of the other changes. |
This comment has been minimized.
This comment has been minimized.
@mkrizek Could you also add ansible/test/sanity/pylint/plugins/blacklist.py Lines 116 to 118 in 124400f
You might want to do it as a separate PR after the fixes are merged. |
@mattclay Sure, will do. |
SUMMARY
Use
from ansible.module_utils.six.moves import shlex_quote
instead.ISSUE TYPE
COMPONENT NAME
many