Remove non-inclusive language #134
Conversation
neonichu
requested review from
abertelrud,
aciidgh and
friedbunny
as code owners
|
@swift-ci please test |
|
We need a corresponding PR on SwiftPM before merging. |
neonichu
force-pushed
the
remove-non-inclusive-language
branch
2 times, most recently
from
30d020d to
836c96e
Compare
|
@swift-ci please test |
rdar://problem/69396992
neonichu
force-pushed
the
remove-non-inclusive-language
branch
from
836c96e to
28ad1ed
Compare
|
@swift-ci please test |
|
@neonichu this is breaking the Windows builds of swift-package-manager since swiftlang/swift-package-manager#2954 isn't merged :-( |
|
Sorry, I had to wait for the final smoke test run to complete. Unfortunately we don't have a good way to merge two PRs together because of how the self-hosted job works. Hopefully we can resolve this type of issue as part of getting rid of the vendored TSC. |
|
Actually, it seems like the real issue is that the self-hosted jobs have been set to "Required". The original intent was apparently to have them for faster CI iteration, but only smoke tests should be required for merging. |
|
OK, this is fixed now. Only the "smoke tests" jobs are required, so in the future we can merge TSC and SwiftPM PRs together. |
rdar://problem/69396992