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

Fix grammar in Readme #47

Merged
merged 1 commit into from
May 13, 2019
Merged

Fix grammar in Readme #47

merged 1 commit into from
May 13, 2019

Conversation

drywet
Copy link
Contributor

@drywet drywet commented May 10, 2019

No description provided.

@@ -143,7 +143,7 @@ Please check out `tethys` package object for all available syntax Ops classes
# derivation

`tethys-derivation` provides semiauto and auto macro derivation JsonReader and JsonWriter instances.
In common case you should prefer semiauto derivation because it's more precise, faster in compile and flexible.
In most cases you should prefer semiauto derivation because it's more precise, faster in compilation and flexible.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not sure about this one.
More substitutions: "in general", "generally", "in general case", "in the general case", "in the common case".

Copy link
Collaborator

Choose a reason for hiding this comment

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

I am ok with current one

@drywet drywet mentioned this pull request May 13, 2019
@eld0727 eld0727 merged commit 34df0a9 into tethys-json:master May 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants