Skip to content

Remove simple json as a required dependency. Install only if json is …#304

Closed
alexpjohnson wants to merge 1 commit intoDataDog:masterfrom
alexpjohnson:remove_simple_json
Closed

Remove simple json as a required dependency. Install only if json is …#304
alexpjohnson wants to merge 1 commit intoDataDog:masterfrom
alexpjohnson:remove_simple_json

Conversation

@alexpjohnson
Copy link
Copy Markdown
Contributor

…not included in the stdlib

Closes #265

@yannmh yannmh self-requested a review November 2, 2018 18:05
@yannmh yannmh mentioned this pull request Nov 2, 2018
@yannmh
Copy link
Copy Markdown

yannmh commented Nov 2, 2018

Thanks a lot @alexpjohnson.
I forked your branch and added an extra commit to remove all references of simplejson. As stated in #265 we don't have anymore reason to use it.

@yannmh
Copy link
Copy Markdown

yannmh commented Nov 2, 2018

Closing this PR in favor of #309

@yannmh yannmh closed this Nov 2, 2018
@yannmh yannmh removed their request for review November 2, 2018 20:21
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.

2 participants