Skip to content

Update ujson dependency - #842

Merged
ccordoba12 merged 6 commits into
palantir:developfrom
steff456:update-ujson
Sep 10, 2020
Merged

Update ujson dependency#842
ccordoba12 merged 6 commits into
palantir:developfrom
steff456:update-ujson

Conversation

@steff456

@steff456steff456 commented Aug 12, 2020

Copy link
Copy Markdown
Contributor

This PR updated the ujson dependency.

Fixes#857
Fixes#802

delta003
delta003 previously approved these changes Sep 7, 2020
@ccordoba12

Copy link
Copy Markdown
Contributor

@delta003, a new release of python-jsonrpc-server is necessary for this one to work.

@delta003

Copy link
Copy Markdown
Contributor

👍 I was hoping to get palantir/python-jsonrpc-server#40 in next release so waiting a day for response

@ccordoba12ccordoba12 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@steff456, one minor comment for you, then this should be ready.

Comment threadsetup.py Outdated
Co-authored-by: Carlos Cordoba <ccordoba12@gmail.com>
This was referenced Sep 10, 2020
@ccordoba12ccordoba12 added this to the 0.35.0 milestone Sep 10, 2020
@ccordoba12ccordoba12 changed the title PR: Update ujson dependencyUpdate ujson dependencySep 10, 2020
@ccordoba12
ccordoba12 merged commit dfdfb09 into palantir:developSep 10, 2020
Sign up for freeto 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.

Pip install fails with ujson error Adapt to ujson > 2.0

3 participants

@steff456@ccordoba12@delta003