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

Add nlohmann_json 3.4.0 for better compatibility #1797

Merged
merged 2 commits into from
Jun 2, 2020
Merged

Add nlohmann_json 3.4.0 for better compatibility #1797

merged 2 commits into from
Jun 2, 2020

Conversation

xyz1001
Copy link
Contributor

@xyz1001 xyz1001 commented Jun 1, 2020

Specify library name and version: nlohmann_json/3.4.0

There are some compatibility issues when I update nlohmann 3.7.x. My team have use jsonformoderncpp/3.4.0@vthiery/stable for nearly two year with no problem. I think this version may has better compatibility.
Here are issues that I had:

@conan-center-bot conan-center-bot added the Bump version PR bumping version without recipe modifications label Jun 1, 2020
@conan-center-bot
Copy link
Collaborator

All green in build 1 (b4dccc567461cc7e2e084e28e20091bc2b15bd66)! 😊

SSE4
SSE4 previously approved these changes Jun 1, 2020
@SSE4 SSE4 requested review from danimtb and uilianries June 1, 2020 07:28
danimtb
danimtb previously approved these changes Jun 1, 2020
@uilianries uilianries dismissed stale reviews from danimtb and SSE4 via 1330515 June 1, 2020 16:42
@uilianries uilianries requested review from SSE4 and danimtb June 1, 2020 16:42
@danimtb danimtb closed this Jun 2, 2020
@danimtb danimtb reopened this Jun 2, 2020
@conan-center-bot
Copy link
Collaborator

All green in build 2 (1330515c271eee2b02a02b48f00571289c3aebc3)! 😊

@danimtb danimtb merged commit 0a6dafb into conan-io:master Jun 2, 2020
@xyz1001 xyz1001 deleted the modify/add_nlohmann_json_3_4_0 branch June 2, 2020 08:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bump version PR bumping version without recipe modifications
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants