Skip to content

ci(dist): ensure pushing to dev-static on stable update#4958

Merged
rami3l merged 1 commit into
rust-lang:mainfrom
rami3l:dist/cd-dev-static-on-stable
Jul 16, 2026
Merged

ci(dist): ensure pushing to dev-static on stable update#4958
rami3l merged 1 commit into
rust-lang:mainfrom
rami3l:dist/cd-dev-static-on-stable

Conversation

@rami3l

@rami3l rami3l commented Jul 13, 2026

Copy link
Copy Markdown
Member

Part of #4738.

During our release process, @Mark-Simulacrum wasn't able to find the required dev-static artifacts from the commit 69d7545 which is probably caused by our new release process disengaging stable from main.

This looks like a pre-existing bug, since the branch used for artifact uploading is semantically incorrect. However, this issue was unfortunately never exposed before since it used to be that stable was tracking main, thus with every stable we would want to release, the artifacts have been uploaded by a previous CI step on main.

This fix ensures that the dev-static uploads happen when stable is updated instead of in addition to main.

@rami3l
rami3l requested review from Mark-Simulacrum and djc July 13, 2026 08:17
Comment thread .github/workflows/ci.yaml Outdated
@rami3l
rami3l force-pushed the dist/cd-dev-static-on-stable branch 6 times, most recently from 9c36b65 to c4fa331 Compare July 13, 2026 15:15
@rami3l
rami3l force-pushed the dist/cd-dev-static-on-stable branch from c4fa331 to d2de939 Compare July 13, 2026 15:16
@rami3l
rami3l requested a review from Mark-Simulacrum July 13, 2026 15:38
@rami3l
rami3l marked this pull request as ready for review July 13, 2026 15:38
@rami3l
rami3l added this pull request to the merge queue Jul 16, 2026
Merged via the queue into rust-lang:main with commit 1bda007 Jul 16, 2026
31 checks passed
@rami3l
rami3l deleted the dist/cd-dev-static-on-stable branch July 16, 2026 20:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants