Code review comment for lp://staging/~freyes/lazr.restfulclient/lp1500460

Revision history for this message
Colin Watson (cjwatson) wrote :

Thanks for the patch. However, I don't think we should do this; the current code doesn't in fact need simplejson on any remotely recent version of Python. I'll see if I can sort out an updated release on PyPI.

If you wanted to produce a modified version of this branch, I think it would be entirely reasonable nowadays to remove the simplejson fallback code paths; that would reduce confusion.

review: Needs Fixing

« Back to merge proposal