Skip to content

chore(ci): enable trusted publishing - #143

Merged
vados-cosmonic merged 1 commit into
bytecodealliance:mainfrom
vados-cosmonic:chore(ci)=enable-trusted-publishing
Mar 5, 2026
Merged

chore(ci): enable trusted publishing#143
vados-cosmonic merged 1 commit into
bytecodealliance:mainfrom
vados-cosmonic:chore(ci)=enable-trusted-publishing

Conversation

@vados-cosmonic

Copy link
Copy Markdown
Collaborator

This commit lightly refactors the release.yml CI flow and enables trusted publishing for NPM.

We are absolutely moving publishing to upstream wasmtime, so this PR might be moot, but given that we no longer need an NPM token any more to make releases, release.yml should still be right for upstream usage.

This commit lightly refactors the `release.yml` CI flow and enables
trusted publishing for NPM

@tschneidereit tschneidereit left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I agree that landing this still makes sense, yes

@vados-cosmonic
vados-cosmonic merged commit 91c466a into bytecodealliance:main Mar 5, 2026
8 checks passed
@vados-cosmonic
vados-cosmonic deleted the chore(ci)=enable-trusted-publishing branch March 5, 2026 10:56
vavsab added a commit to vavsab/weval that referenced this pull request Jul 9, 2026
Enables npm trusted publishing for `@bytecodealliance/weval`, following
the same pattern used by wizer (bytecodealliance/wizer#143).

- `release.yml`: add `permissions: {}` at top level, scope per-job
  permissions, add `actions/setup-node` with `registry-url` (enables
  OIDC auth — no NPM_TOKEN needed), and publish on `v*` tag pushes.
- `npm/weval/update.js`: updates `package.json` version and the TAG
  constant in `index.js` before publishing.
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