Skip to content

Fix typos and grammar issues in comments#483

Merged
Syfaro merged 1 commit intogo-telegram-bot-api:masterfrom
alexandear:fix-typos-grammar
Nov 8, 2021
Merged

Fix typos and grammar issues in comments#483
Syfaro merged 1 commit intogo-telegram-bot-api:masterfrom
alexandear:fix-typos-grammar

Conversation

@alexandear
Copy link
Copy Markdown
Contributor

This PR fixes issues in comments:

  • Typos in words.
  • Use a comma before 'so' if it connects two independent clauses (unless they are closely connected and short). 
  • This word is normally spelled with a hyphen.
  • Use a comma before 'or' if it connects two independent clauses (unless they are closely connected and short). 
  • Use an instead of 'a' if the following word starts with a vowel sound, e.g. 'an article', 'an hour'. 
  • Did you mean past? 
  • Consider using either the past participle filed or the present participle filing here. 
  • Consider using many. 
  • When '64-bit' is used as a modifier, it is usually spelled with a hyphen. 
  • The past participle is required after "must be", alternatively you could omit the "be". 

Copy link
Copy Markdown
Member

@Syfaro Syfaro left a comment

Choose a reason for hiding this comment

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

Mostly looks good, I think there's one thing that wasn't quite correct.

I also quite substantially updated the master branch, so it might take a little bit of messing around to get it into a state where it can be merged again, sorry!

Comment thread params.go Outdated
@alexandear
Copy link
Copy Markdown
Contributor Author

@Syfaro I updated PR, please recheck

Copy link
Copy Markdown
Member

@Syfaro Syfaro 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, thanks!

@Syfaro Syfaro merged commit 8999cea into go-telegram-bot-api:master Nov 8, 2021
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.

2 participants