Skip to content

Conversation

@rnro
Copy link
Contributor

@rnro rnro commented Mar 3, 2025

Use nightly_next as swift version

see #3122

Motivation:
To not have to rename threshold directories when the nightly branch
changes.

Modifications:

  • Use nightly_next as swift version in the matrix generation script which
    is picked up by the benchmark script.
  • Move nightly-next thresholds and add legacy symlink

Result:

Benchmark thresholds will attempt to find directories named
nightly_next, not nightly_6_1.

rnro added 2 commits March 3, 2025 15:47
Motivation:
To not have to rename threshold directories when the nightly branch
changes.

Modifications:
Use nightly_next as swift version in the matrix generation script which
is picked up by the benchmark script.

Result:

Benchmark thresholds will attempt to find directories named
nightly_next, not nightly_6_1.
@rnro rnro added the semver/none No version bump required. label Mar 3, 2025
@Lukasa Lukasa enabled auto-merge (squash) March 3, 2025 15:52
@rnro rnro disabled auto-merge March 3, 2025 15:53
@rnro rnro enabled auto-merge (squash) March 3, 2025 15:54
@rnro rnro merged commit 517dcca into main Mar 4, 2025
30 of 35 checks passed
@rnro rnro deleted the rename_nightly_6_1_to_nightly_next branch March 4, 2025 09:37
rnro added a commit to rnro/swift-nio that referenced this pull request Mar 7, 2025
Following on from apple#3126 delete `Benchmarks/Thresholds/nightly-6.1` and `IntegrationTests/tests_04_performance/Thresholds/nightly-6.1.json` which is no longer needed now that the shared benchmarks workflow has been updated.
Lukasa pushed a commit that referenced this pull request Mar 7, 2025
Following on from #3126 delete
`Benchmarks/Thresholds/nightly-6.1` and
`IntegrationTests/tests_04_performance/Thresholds/nightly-6.1.json`
which is no longer needed now that the shared benchmarks workflow has
been updated.
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