Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix inappropriate lanugage in NIO #1554

Merged
merged 1 commit into from
Jun 12, 2020
Merged

Conversation

weissi
Copy link
Member

@weissi weissi commented Jun 12, 2020

Motivation:

It's 2020, we shouldn't use unnecessary offensive, insulting, or
outdated language.

Modifications:

Fix a few examples.

Result:

More inclusive.

Motivation:

It's 2020, we shouldn't use unnecessary offensive, insulting, or
outdated language.

Modifications:

Fix a few examples.

Result:

More inclusive.
@weissi weissi requested review from Lukasa, tomerd and glbrntt June 12, 2020 17:49
Copy link
Contributor

@Lukasa Lukasa left a comment

Choose a reason for hiding this comment

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

100%, great patch

Copy link
Member

@tomerd tomerd left a comment

Choose a reason for hiding this comment

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

<3

@Lukasa Lukasa merged commit 5268726 into apple:master Jun 12, 2020
pull bot pushed a commit to scope-demo/swift-nio that referenced this pull request Jun 12, 2020
Motivation:

It's 2020, we shouldn't use unnecessary offensive, insulting, or
outdated language.

Modifications:

Fix a few examples.

Result:

More inclusive.
@PeterAdams-A PeterAdams-A added the 🔨 semver/patch No public API change. label Jun 22, 2020
@Lukasa Lukasa added this to the 2.19.0 milestone Jun 23, 2020
Copy link
Member

@ktoso ktoso left a comment

Choose a reason for hiding this comment

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

<3

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
🔨 semver/patch No public API change.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants