tidwall / sjson

Set JSON values very quickly in Go
MIT License
2.41k stars 165 forks source link

Some grammar changes on README #2

Closed ghost closed 7 years ago

ghost commented 7 years ago

Not that my english are better than yours but I want to help as much as I can to welcome you, nice job with this library, don't stop, keep going and good luck!

tidwall commented 7 years ago

Thanks for the PR. I haven't decided if this warrants a merge just yet, but I noticed "unknown repository":

image

I only merge through the command line and I can't find your fork on Github. I prefer merging against traceable forks.

ghost commented 7 years ago

It's just one button, what are you scary for it's not a virus :)

I don't make prs to approve them, just look the changes and learn from them, do these by your own if you want without merge, I am not like others :) Good luck!!

tidwall commented 7 years ago

thanks for the response.