Skip to content

ci: Use shared workspace release workflow#938

Merged
huitseeker merged 2 commits into
nextfrom
ci/use-reusable-workspace-release
Apr 30, 2026
Merged

ci: Use shared workspace release workflow#938
huitseeker merged 2 commits into
nextfrom
ci/use-reusable-workspace-release

Conversation

@huitseeker
Copy link
Copy Markdown
Collaborator

@huitseeker huitseeker commented Apr 8, 2026

This switches the dry-run and publish workflows to the shared release workflow in 0xMiden/.github : 0xMiden/.github#11

It passes scripts/check-msrv.sh explicitly and removes the old local release action.

Removed unnecessary OIDC permissions from the workspace release dry-run caller while keeping publish permissions unchanged.

Note for @bobbinth : This operation might (but should not, from what I could gather) require changing the trusted publishing workflow configured for the p3-miden and crypto crates (that should be set to the caller workflow right now, and might need setting to the callee).

@huitseeker huitseeker added the no changelog This PR does not require an entry in the `CHANGELOG.md` file label Apr 8, 2026
@huitseeker huitseeker marked this pull request as draft April 8, 2026 10:47
@huitseeker huitseeker force-pushed the ci/use-reusable-workspace-release branch 9 times, most recently from 64dbeda to 4bc66a0 Compare April 13, 2026 12:30
@huitseeker huitseeker force-pushed the ci/use-reusable-workspace-release branch from 4bc66a0 to 526594e Compare April 24, 2026 19:24
@huitseeker huitseeker changed the title ci: Use shared workspace release workflow (do not merge) ci: Use shared workspace release workflow Apr 24, 2026
@huitseeker huitseeker marked this pull request as ready for review April 24, 2026 19:35
Copy link
Copy Markdown
Contributor

@bobbinth bobbinth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Thank you!

Note for @bobbinth : This operation might (but should not, from what I could gather) require changing the trusted publishing workflow configured for the p3-miden and crypto crates (that should be set to the caller workflow right now, and might need setting to the callee).

I guess the path forward here is to just try to do the release, and if it doesn't work, update the settings - right?

@huitseeker
Copy link
Copy Markdown
Collaborator Author

I guess the path forward here is to just try to do the release, and if it doesn't work, update the settings - right?

Correct.

@huitseeker huitseeker merged commit 509ee47 into next Apr 30, 2026
29 checks passed
@huitseeker huitseeker deleted the ci/use-reusable-workspace-release branch April 30, 2026 20:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no changelog This PR does not require an entry in the `CHANGELOG.md` file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants