Skip to content

Conversation

@dhalbert
Copy link
Contributor

We're renaming ujson to json in 4.0.0. Tries to import json parser as either json or ujson for upward compatibility. Renames import to json_module to avoid conflicts with variables named json.

Copy link
Collaborator

@makermelissa makermelissa left a comment

Choose a reason for hiding this comment

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

Code looks good. I won’t be able to actually test until tonight though.

@makermelissa
Copy link
Collaborator

Ok tested good. It looks like it's working.

@jerryneedell jerryneedell merged commit 08c1726 into adafruit:master Feb 22, 2019
adafruit-adabot added a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request Feb 22, 2019
@dhalbert dhalbert deleted the ujson-to-json branch May 28, 2022 12:49
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.

3 participants