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 re-usable workflows to use nightly_next #3122

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

rnro
Copy link
Contributor

@rnro rnro commented Feb 25, 2025

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 parameters to refer to nightly_next rather than nightly_6_1, implement
a shim so that the old parameter names still work.

Result:

We will be able to modify top-level workflows to use nightly_next
terminology.

Test run with an additional commit opting in to use these changes at the top level https://github.com/apple/swift-nio/actions/runs/13524774485/job/37792437713?pr=3122

rnro added 2 commits February 25, 2025 15:23
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 parameters 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 top-level workflows to use nightly_next
terminology.
@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 16:22
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.

1 participant