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

Update generate matrix stript to use nightly_next #3120

Merged
merged 1 commit into from
Feb 25, 2025
Merged

Update generate matrix stript to use nightly_next #3120

merged 1 commit into from
Feb 25, 2025

Conversation

rnro
Copy link
Contributor

@rnro rnro commented Feb 25, 2025

Update generate matrix stript to use nightly_next

Motivation:

Updating the CI parameters for changing nightly job targets with each new Swift version (6.0, 6.1, ...) is onerous. Whilst this gives us some value in the non-nightly pipelines by forcing adopters to apply thought to what is applied to each supported version, the nightlies arguably deliver less value in this regard as they tend to be non-required.

Modifications:

Change flags to refer to nightly_next rather than nightly_6_1, implement a shim so that the old environment variables still work.

Result:

We will be able to modify adopter workflows to use nightly_next terminology.

Example of a job where workflows are using the new matrix https://github.com/apple/swift-nio/actions/runs/13520016728

@rnro rnro force-pushed the nightly_next branch 7 times, most recently from 440fe0f to d32d489 Compare February 25, 2025 11:20
Motivation:

Updating the CI parameters for changing nightly job targets with each
new Swift version (6.0, 6.1, ...) is onerous. Whilst this gives us some
value in the non-nightly pipelines by forcing adopters to apply thought
to what is applied to each supported version, the nightlies arguably
deliver less value in this regard as they tend to be non-required.

Modifications:

Change flags to refer to nightly_next rather than nightly_6_1, implement
a shim so that the old environment variables still work.

Result:

We will be able to modify adopter workflows to use nightly_next
terminology.
@rnro rnro changed the title testing Update generate matrix stript to use nightly_next Feb 25, 2025
@rnro rnro added the semver/none No version bump required. label Feb 25, 2025
@rnro rnro marked this pull request as ready for review February 25, 2025 13:34
@rnro rnro merged commit 4c14ce5 into main Feb 25, 2025
33 of 36 checks passed
@rnro rnro deleted the nightly_next branch February 25, 2025 13:58
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.

2 participants