Skip to content

fix(ci): remove environment from publish workflow#23

Merged
jasonz-dfinity merged 1 commit intomainfrom
fix/publish-remove-environment
Feb 26, 2026
Merged

fix(ci): remove environment from publish workflow#23
jasonz-dfinity merged 1 commit intomainfrom
fix/publish-remove-environment

Conversation

@jasonz-dfinity
Copy link
Contributor

@jasonz-dfinity jasonz-dfinity commented Feb 26, 2026

Summary

  • Remove environment: release from the publish workflow to match the crates.io trusted publisher configuration, which does not specify an environment
  • This mismatch caused the v1.1.2 publish to fail with: No Trusted Publishing config found for repository dfinity/metrics-encoder
  • The environment: release doesn't provide any benefit since the repo's write access is protected already

After merging

Re-run the failed v1.1.2 release workflow, or delete and re-create the release to re-trigger publishing.

@jasonz-dfinity jasonz-dfinity requested review from a team as code owners February 26, 2026 22:09
The crates.io trusted publisher config for this crate does not specify
an environment, so the workflow must not specify one either. This
mismatch caused the OIDC token exchange to fail with a 400 error.
@jasonz-dfinity jasonz-dfinity force-pushed the fix/publish-remove-environment branch from 3f7395f to c2a9465 Compare February 26, 2026 22:11
@jasonz-dfinity jasonz-dfinity merged commit a43e6d2 into main Feb 26, 2026
6 checks passed
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.

2 participants