Skip to content

Fix typo in the documentation #12

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

Closed
wants to merge 1 commit into from
Closed

Fix typo in the documentation #12

wants to merge 1 commit into from

Conversation

yamadapc
Copy link

@yamadapc yamadapc commented Nov 8, 2014

"these data have to be" was changed to "the data has to be"

@hyperthunk
Copy link
Member

Data is the plural of Datum, so there's no typo there! Do you think it stands out as wrong given the general mis-use of the word data? I don't mind changing it, but grammatically there's nothing wrong with "these data".

@yamadapc
Copy link
Author

Oh... I'm sorry, I didn't know that. My bad. It stands out a bit to me, but I guess it's because I'm not a native english speaker. I don't think there's any need to change it.

@yamadapc yamadapc closed this Nov 13, 2014
@yamadapc
Copy link
Author

I thought it was a typo, because it followed both "messages" and "Int, String, Ping, Pong, etc", which would be referred to in plural. But that makes sense; I can't tell what is more suitable for the context of this document.

@yamadapc yamadapc deleted the patch-1 branch November 13, 2014 11:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants