Skip to content

Updated year in README.md#1425

Merged
nlohmann merged 1 commit into
nlohmann:developfrom
jef:patch-1
Jan 12, 2019
Merged

Updated year in README.md#1425
nlohmann merged 1 commit into
nlohmann:developfrom
jef:patch-1

Conversation

@jef

@jef jef commented Jan 11, 2019

Copy link
Copy Markdown
Contributor

Updates the README from 2018 to 2019

Happy New Year 🎉

Pull request checklist

Read the Contribution Guidelines for detailed information.

  • Changes are described in the pull request, or an existing issue is referenced.
  • The test suite compiles and runs without error.
  • Code coverage is 100%. Test cases can be added by editing the test suite.
  • The source code is amalgamated; that is, after making changes to the sources in the include/nlohmann directory, run make amalgamate to create the single-header file single_include/nlohmann/json.hpp. The whole process is described here.

@coveralls

coveralls commented Jan 11, 2019

Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 100.0% when pulling 7f73915 on hijxf:patch-1 into df460c9 on nlohmann:develop.

@nickaein

Copy link
Copy Markdown
Contributor

There is another 2018 reference in very last lines of README.md in copyright note. Is there any reason to update/not update it?

There is also other references to 2018 throughout the code that should be considered.

@nlohmann

Copy link
Copy Markdown
Owner

No need to update the copyright years. I will do this with the next release.

@nlohmann nlohmann left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Looks good to me.

@jef

jef commented Jan 11, 2019

Copy link
Copy Markdown
Contributor Author

Awesome. Thanks!

@jef jef closed this Jan 11, 2019
@jef jef reopened this Jan 11, 2019
@jef

jef commented Jan 11, 2019

Copy link
Copy Markdown
Contributor Author

Accidentally closed.

Thanks,
Jef

@nlohmann nlohmann merged commit b39f34e into nlohmann:develop Jan 12, 2019
@nlohmann

Copy link
Copy Markdown
Owner

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants