Skip to content
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

Use welcoming language #274

Merged
merged 1 commit into from
Jan 22, 2021
Merged

Conversation

fabianfett
Copy link
Member

To comply with the new Swift code of conduct, let's make the language a little more welcoming.

Modifications:

  • Replaced usages of "sanity" with "soundness"
  • Added "sanity" to the list of unacceptable terms.

Result:

  • Swift NIO HTTP2 is nicer.

@swift-server-bot
Copy link

Can one of the admins verify this patch?

8 similar comments
@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@Lukasa
Copy link
Contributor

Lukasa commented Jan 22, 2021

@swift-nio-bot add to whitelist

@Lukasa Lukasa added the semver/none No version bump required. label Jan 22, 2021
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.

Generally fine modulo concerns with CI.

@@ -22,9 +22,9 @@ services:
- CAP_NET_RAW
- CAP_NET_BIND_SERVICE

sanity:
Copy link
Contributor

Choose a reason for hiding this comment

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

As with NIO, I believe this change requires coordination with @tomerd in order to ensure the CI continues to function effectively.

@Lukasa
Copy link
Contributor

Lukasa commented Jan 22, 2021

Oh, it seems that @tomerd has already made that change comprehensively across our CI systems. I suspect this may actually have broken some repos.

@Lukasa Lukasa merged commit 16e49c6 into apple:main Jan 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver/none No version bump required.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants