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.

Sign up for GitHub

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

Jump to bottom

📝 fixing documentation #867 #868

Merged
merged 2 commits into from
Dec 11, 2017
Merged

📝 fixing documentation #867 #868

merged 2 commits into from
Dec 11, 2017

Conversation

nlohmann
Copy link
Owner

@nlohmann nlohmann commented Dec 10, 2017 •
edited
Loading

Fix for #867

Pull request checklist

@nlohmann
📝 fixing documentation #867
772bb3c 
The example in the documentation on how to "force" an array of arrays was wrong since the first release. Fixed the documentation and added checks for the README unit tests.
@nlohmann nlohmann self-assigned this Dec 10, 2017
@nlohmann nlohmann added this to the Release 3.0.0 milestone Dec 10, 2017
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 7de009e on feature/issue867 into 0693945 on develop.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 7de009e on feature/issue867 into 0693945 on develop.

@nlohmann nlohmann merged commit 383743c into develop Dec 11, 2017
@nlohmann nlohmann deleted the feature/issue867 branch December 11, 2017 07:56
This was referenced Dec 11, 2017
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@nlohmann nlohmann

Labels
documentation
Projects
None yet
Milestone
Release 3.0.0
Development

Successfully merging this pull request may close these issues.
2 participants
@nlohmann @coveralls