Skip to content

Conversation

euanh
Copy link
Collaborator

@euanh euanh commented Sep 22, 2025

Motivation

Swift NIO's unit test matrix job has been updated for Swift 6.2, so additional override flags must be defined.

Modifications

Define linux_6_2_arguments_override for pull requests and the scheduled build on main.

Result

Tests with Swift 6.2 will pass.

Test Plan

Existing tests pass again.

@euanh euanh force-pushed the swift-6.2-overrides branch from 948c0d7 to 52261b7 Compare September 22, 2025 09:37
Motivation
----------

Swift NIO's unit test matrix job has been [updated for Swift
6.2](apple/swift-nio#3374),
so additional override flags must be defined.

Modifications
-------------

Define `linux_6.2_arguments_override` for pull requests and the
scheduled build on main.

Result
------

Tests with Swift 6.2 will pass.

Test Plan
---------

Existing tests pass again.
@euanh euanh force-pushed the swift-6.2-overrides branch from 52261b7 to 7f57d1e Compare September 22, 2025 09:39
@euanh euanh added semver/none No version bump required. kind/administration CI, repo automation, other admin tasks labels Sep 22, 2025
@euanh
Copy link
Collaborator Author

euanh commented Sep 22, 2025

Overriding the failing 6.2 test: GitHub won't pick up the linux_6_2_arguments_override change until this PR is merged.

@euanh euanh merged commit a87a0b1 into apple:main Sep 22, 2025
46 of 48 checks passed
@euanh euanh deleted the swift-6.2-overrides branch September 22, 2025 10:14
euanh added a commit that referenced this pull request Sep 25, 2025
Motivation
----------

#158 added Swift 6.2 flags to the `main` workflow, but the
`pull_request` workflow has a separate list of flags which was not
updated.

Modifications
-------------

Update `pull_request` workflow to match `main`.

Result
------

Correct 6.2 flags will be applied to pull request workflow runs.

Test Plan
---------

This PR will fail because the integration and end-to-end tests does
still install the 6.1 Static SDK.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/administration CI, repo automation, other admin tasks semver/none No version bump required.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant