Skip to content

ci: use cargo publish --dry-run to verify packaged crates#32

Open
Firestar99 wants to merge 1 commit intomainfrom
publish-dry-run
Open

ci: use cargo publish --dry-run to verify packaged crates#32
Firestar99 wants to merge 1 commit intomainfrom
publish-dry-run

Conversation

@Firestar99
Copy link
Copy Markdown
Member

@Firestar99 Firestar99 commented Apr 10, 2026

  • use cargo package instead of release-plz to verify packaged crates
  • release-plz's dry-run doesn't work properly and randomly fails:

release-plz dry-run problems

Half of the time, it succeeds with Warning: release-plz release-pr failed with HTTP 422; continuing with no PRs created.:

And the other half it just fails with:

ERROR failed to create ref refs/heads/release-plz-2026-04-09T18-06-58Z with sha 87c458454a0f5222e7d4d1bb8621a8aec8e28bdb

Caused by:
    0: Response body:
       {
         "documentation_url": "https://docs.github.com/rest/git/refs#create-a-reference",
         "message": "Resource not accessible by integration",
         "status": "403"
       }
    1: HTTP status client error (403 Forbidden) for url (https://api.github.com/repos/Rust-GPU/spirv-tools-rs/git/refs)
Error: failed to create ref refs/heads/release-plz-2026-04-09T18-06-58Z with sha 87c458454a0f5222e7d4d1bb8621a8aec8e28bdb

Rereading the documentation, it does state:

the --dry-run flag is only added to the release command (the flag isn't added to the release-pr command).

@Firestar99
Copy link
Copy Markdown
Member Author

Firestar99 commented Apr 10, 2026

This action should have failed due to the packaged release not compiling, but didn't, proving that release-plz release --dry-run does not run cargo package with verification.

@Firestar99 Firestar99 mentioned this pull request Apr 10, 2026
@Firestar99 Firestar99 force-pushed the publish-dry-run branch 2 times, most recently from 399a9b4 to 3c8320e Compare April 10, 2026 12:30
@Firestar99 Firestar99 changed the title make publish dry run consistent ci: switch release-dry-run to cargo package for consistency Apr 10, 2026
@Firestar99 Firestar99 changed the title ci: switch release-dry-run to cargo package for consistency ci: use cargo publish --dry-run to verify packaged crates Apr 10, 2026
@Firestar99 Firestar99 requested a review from a team April 10, 2026 12:45
@Firestar99 Firestar99 marked this pull request as ready for review April 10, 2026 12:45
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.

1 participant