Skip to content

Small grammar change to pull requests guide - #4322

Merged
Travis Plunk (TravisEz13) merged 1 commit into
masterfrom
zjalexander-patch-1
Aug 4, 2017
Merged

Small grammar change to pull requests guide#4322
Travis Plunk (TravisEz13) merged 1 commit into
masterfrom
zjalexander-patch-1

Conversation

@zjalexander

Copy link
Copy Markdown
Contributor

him -> them, which is consistent with usage above.

him -> them, which is consistent with usage above.
@msftclas

Zachary Alexander (@zjalexander),
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@TravisEz13 Travis Plunk (TravisEz13) left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Them is more appropriate than him but perhaps we should adopt a writing style to guide this. Here is a link to guidance on resolving this for APA Style

@TravisEz13

Travis Plunk (TravisEz13) commented Jul 24, 2017

Copy link
Copy Markdown
Member

Please fix PR title
Please use the present tense, imperative style for commit messages and title per Contribution Guide
Suggestion Remove gender specific language in PR process

@zjalexander Zachary Alexander (zjalexander) changed the title style change Small grammar change to pull requests guide Jul 24, 2017
@TravisEz13
Travis Plunk (TravisEz13) merged commit 7d5367a into master Aug 4, 2017
@TravisEz13
Travis Plunk (TravisEz13) deleted the zjalexander-patch-1 branch August 4, 2017 21:59
Thatgfsj (Thatgfsj) pushed a commit to Thatgfsj/PowerShell that referenced this pull request Aug 6, 2026
him -> them, which is consistent with usage above.
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.

3 participants