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

A JSONEncoder class that uses ujson to encode #347

Open
retnikt opened this issue Feb 9, 2020 · 3 comments
Open

A JSONEncoder class that uses ujson to encode #347

retnikt opened this issue Feb 9, 2020 · 3 comments

Comments

@retnikt
Copy link

retnikt commented Feb 9, 2020

Sort of the exact converse of #124. It would be nice to have a JSONEncoder class that uses ujson for encoding built-in. At the moment I have to write a wrapper:

class UJSONEncoder(JSONEncoder):
    encode = ujson.encode

Big deal, I know, but this would make things easier. I would just open a PR for this, but at the moment ujson has no pure-python wrapper module where this could be defined so a slight package architectural change would be needed?

@hnykda
Copy link

hnykda commented Feb 14, 2020

This project is dead. I recommend you to switch to something else, like orjson

@takos22
Copy link

takos22 commented May 20, 2021

This would be a nice feature to have to better work as a json drop-in replacement.

This project is dead. I recommend you to switch to something else, like orjson

By the way, @hnykda it isn't dead anymore, see #343 (comment) 🎉

@hnykda
Copy link

hnykda commented May 20, 2021

Yep, happy to see this happening 👍 . That comment was written before it was picked up again

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

No branches or pull requests

3 participants