Skip to content

Add release workflow with npm trusted publishing#3

Merged
ivoIturrieta merged 1 commit into
mainfrom
feat/publish-workflow
May 23, 2026
Merged

Add release workflow with npm trusted publishing#3
ivoIturrieta merged 1 commit into
mainfrom
feat/publish-workflow

Conversation

@ivoIturrieta
Copy link
Copy Markdown
Collaborator

Mirrors the elements publish flow: PR merged to main triggers build & test, then a reviewer approves the publish via the npm-publish environment gate. Version is read from npm and bumped from there (patch by default, override with release:minor or release:major PR labels). Auth uses OIDC trusted publishing — no NPM_TOKEN.

Also corrects package.json version from 1.0.0 to 0.1.0 to match what is actually published on npm.

Copy link
Copy Markdown
Member

@brunolemos brunolemos left a comment

Choose a reason for hiding this comment

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

Please use node 24 (LTS) in any new code

Copy link
Copy Markdown
Member

@brunolemos brunolemos left a comment

Choose a reason for hiding this comment

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

Also pnpm 10 or 11 but that's not a blocker so approving.
You could rebase with main to fix merge conflicts and keep all this in a single commit.

Mirrors the elements publish flow: PR merged to main triggers build & test,
then a reviewer approves the publish via the npm-publish environment gate.
Version is read from npm and bumped from there (patch by default, override
with release:minor or release:major PR labels). Auth uses OIDC trusted
publishing — no NPM_TOKEN.

Also bumps ci.yml + package.json engines to Node 24 (LTS) and pnpm 10
to match the publish workflow.
@ivoIturrieta ivoIturrieta force-pushed the feat/publish-workflow branch from 5ca7ddb to 1f7b969 Compare May 23, 2026 17:38
@ivoIturrieta ivoIturrieta merged commit ca7f177 into main May 23, 2026
1 check 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