Conversation
|
The release job now runs inside a Before this is merged, the
Note that an environment that is referenced but never configured is created automatically with no protection rule at all, so this step is not optional. |
| permissions: | ||
| id-token: write # required for npm provenance via OIDC | ||
| contents: write # required to push the release commit and the tag | ||
| uses: fastify/workflows/.github/workflows/reusable-release.yml@7d11b7f |
Tony133
left a comment
There was a problem hiding this comment.
This is blocked on fastify/workflows#237: uses: points to a PR commit (7d11b7f, already behind the head of fastify/workflows#237).
Next steps: once fastify/workflows#237 is released, repin to the tag (or full SHA) and we're good. On our side we also need the release environment in this repo (fastify/release as required reviewer, no self-review, deployments only from main) and fastify-cli set up as npm trusted publisher, otherwise the gate does nothing.
What
Adds a
releaseworkflow that publishesfastify-clito npm from the Actions tab: pick the bump type (patch/minor/major) and the reusable workflow does the rest.It delegates to the new reusable workflow proposed in fastify/workflows#237, which bumps the version, commits it as
Bumped vX.Y.Z, publishes to npm with provenance and creates the GitHub release with generated notes.Blocked on
uses:reference is currently pinned to that PR's commit SHA and must be repinned to the resulting release tag before this PR is merged (marked with aTODOin the file).fastify-climust be configured as an npm trusted publisher, since the publish authenticates via OIDC rather than anNPM_TOKENsecret.Notes
The release workflow does not run the test suite; CI already validates every commit on
main. There is norelease:buildscript in this package, so that step is a no-op here.