-
Notifications
You must be signed in to change notification settings - Fork 11
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
Change "I" to "We" to make it sound like a team project #265
Conversation
Check for wordiness and weird English rules that linguist care about.
This doesn't make a difference to me, but should we avoid personal pronouns ("you", "I", etc.)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These are good stylistic changes and in fact please continue to make any changes along these lines w/o seeking a review (just merge these types of changes). I trust your judgement on these. Great work!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In addition to my remarks, I would like that "one sentence per row" rule would be applied (I see that in some places several sentences per row are used). It makes easier to track and review the text in version control system.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes made
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Got it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One more remark: the branch should have been named as feature/vign-edits
and not vign-edits
.
It's enough changes for a single PR. Merging it. |
This is an example of PR as suggested by the venerable @GegznaV and @bryanhanson