-
Notifications
You must be signed in to change notification settings - Fork 23.8k
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
Remove use of simplejson throughout code base #43548
Conversation
# Version 1.5 released 2007-01-18 does not have the encoding parameter which we need | ||
print('\n{"msg": "Error: Ansible requires the stdlib json or simplejson >= 1.6. Neither was found!", "failed": true}') | ||
print('\n{"msg": "Error: ansible requires the stdlib json and was not found!", "failed": true}') | ||
sys.exit(1) | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not entirely sure what we want to do with these... The libs which override json are still present on pypi. But I bet nowadays they will break more than ansible so we don't have to worry about them. On the other hand, I would have thought that when I had to add the code that handled that.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've added back in the python-json checks.
This looks good +1 |
bot_skip For > 50 files |
Build succeeded (third-party-check pipeline).
|
SUMMARY
Remove use of simplejson throughout code base. Fixes #42761
ISSUE TYPE
COMPONENT NAME
many
ANSIBLE VERSION
ADDITIONAL INFORMATION