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

Typos README #811

Merged
merged 1 commit into from
Oct 31, 2017
Merged

Typos README #811

merged 1 commit into from
Oct 31, 2017

Conversation

Itja
Copy link
Contributor

@Itja Itja commented Oct 31, 2017

Hey, thanks for this great library which I've used in many occasions now. I know it's not much, but I wanted to contribute at least a tiny bit back to you by this PR :-) Keep up the good work!

Hey, thanks for this great library which I've used in many occasions now. I know it's not much, but I wanted to contribute at least a tiny bit back to you by this PR :-) Keep up the good work!
@nlohmann nlohmann added this to the Release 3.0.0 milestone Oct 31, 2017
@nlohmann nlohmann self-assigned this Oct 31, 2017
Copy link
Owner

@nlohmann nlohmann left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@trilogy-service-ro
Copy link

Can one of the admins verify this patch?

@nlohmann nlohmann merged commit 1af5601 into nlohmann:develop Oct 31, 2017
@nlohmann
Copy link
Owner

Thanks a lot!

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling e423aea on Itja:patch-1 into 5696660 on nlohmann:develop.

@Itja Itja deleted the patch-1 branch November 2, 2017 10:54
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