-
Notifications
You must be signed in to change notification settings - Fork 168
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
Bump bundled nlohmann::json to commit 84f19d6 #209
Bump bundled nlohmann::json to commit 84f19d6 #209
Conversation
There is a gcc10 compilation issue that got resolved in [acts-project#2034](nlohmann/json#2034). There has not been a tag since this was merged so I'm bumping the bundled version to the develop branch immediately after that PR went in.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Uncontroversial
Codecov Report
@@ Coverage Diff @@
## master #209 +/- ##
=======================================
Coverage 44.88% 44.88%
=======================================
Files 369 369
Lines 18451 18451
Branches 8737 8737
=======================================
Hits 8281 8281
Misses 4861 4861
Partials 5309 5309 Continue to review full report at Codecov.
|
@paulgessinger Could you rebase? |
I can but it should b safe to do a squash and merge in this case. |
Yes, true. |
There is a gcc10 compilation issue that got resolved in [acts-project#2034](nlohmann/json#2034). There has not been a tag since this was merged so I'm bumping the bundled version to the develop branch immediately after that PR went in.
There is a gcc10 compilation issue that got resolved in
#2034. There has not been
a tag since this was merged so I'm bumping the bundled version to the
develop branch immediately after that PR went in.