Skip to content

Commit eb797c5

Browse files
committed
Enable the unacceptable language soundness check and resolve identified issues
1 parent 77f84d3 commit eb797c5

File tree

7 files changed

+6
-7
lines changed

7 files changed

+6
-7
lines changed

.github/workflows/pull_request.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,5 @@ jobs:
2424
with:
2525
license_header_check_project_name: "Swift"
2626
docs_check_enabled: false
27-
unacceptable_language_check_enabled: false
2827
format_check_enabled: false
2928
api_breakage_check_enabled: false

Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
FROM swiftlang/swift:nightly-main-jammy
1212

1313
# Set up the current build user in the same way done in the Swift.org CI system:
14-
# https://github.com/swiftlang/swift-docker/blob/main/swift-ci/master/ubuntu/22.04/Dockerfile
14+
# https://github.com/swiftlang/swift-docker/blob/main/swift-ci/main/ubuntu/22.04/Dockerfile
1515

1616
RUN groupadd -g 998 build-user && \
1717
useradd -m -r -u 998 -g build-user build-user

Sources/Testing/ExitTests/ExitTest.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -263,7 +263,7 @@ extension ExitTest {
263263
// code 3 [...]".
264264
//
265265
// The Wine project's implementation of raise() calls `_exit(3)` by default.
266-
// See https://github.com/wine-mirror/wine/blob/master/dlls/msvcrt/except.c
266+
// See https://github.com/wine-mirror/wine/blob/master/dlls/msvcrt/except.c (ignore-unacceptable-language)
267267
//
268268
// Finally, an official copy of the UCRT sources (not up to date) is hosted
269269
// at https://www.nuget.org/packages/Microsoft.Windows.SDK.CRTSource . That

Sources/Testing/ExitTests/SpawnProcess.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -224,7 +224,7 @@ func spawnExecutable(
224224
}
225225
#if SWT_TARGET_OS_APPLE && DEBUG
226226
// Resume the process.
227-
_ = kill(pid, SIGCONT)
227+
_ = kill(pid, SIGCONT) // ignore-unacceptable-language
228228
#endif
229229
return pid
230230
}

Sources/Testing/Testing.docc/LimitingExecutionTime.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ Some tests may naturally run slowly: they may require significant system
1818
resources to complete, may rely on downloaded data from a server, or may
1919
otherwise be dependent on external factors.
2020

21-
If a test may hang indefinitely or may consume too many system resources to
21+
If a test may stall indefinitely or may consume too many system resources to
2222
complete effectively, consider setting a time limit for it so that it's marked
2323
as failing if it runs for an excessive amount of time. Use the
2424
``Trait/timeLimit(_:)-4kzjp`` trait as an upper bound:

Tests/TestingTests/IssueTests.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -484,7 +484,7 @@ final class IssueTests: XCTestCase {
484484
}
485485

486486
func testCastAsAnyProtocol() async {
487-
// Sanity check that we parse types cleanly.
487+
// Check that we parse types cleanly.
488488
await Test {
489489
#expect((1 as Any) is any Numeric)
490490
_ = try #require((1 as Any) as? any Numeric)

Tests/TestingTests/Support/CartesianProductTests.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ struct CartesianProductTests {
3737

3838
@Test("First element is correct")
3939
func firstElement() throws {
40-
// Sanity-check the first element is correct. (This value is also tested in
40+
// Check that the first element is correct. (This value is also tested in
4141
// testCompleteEquality().)
4242
let (c1, c2, product) = computeCartesianProduct()
4343
let first = try #require(product.first(where: { _ in true }))

0 commit comments

Comments
 (0)