Skip to content

Conversation

@mansona
Copy link
Member

@mansona mansona commented Feb 7, 2026

Our last attempt to release an alpha failed because our npm token has now expired 😭

I have already setup OIDC and this PR runs pnpm create release-plan-setup@latest to update our release-plan CI jobs to align with the new defaults and make it possible to release again

Note: we can no longer have multiple publish-branchname.yml CI jobs because of OIDC, I will temporarily go back to maintaining changes across branches on this file but I have a plan to make this easier to manage going forward

@mansona mansona requested a review from a team February 7, 2026 16:30
Copy link
Contributor

@NullVoxPopuli NullVoxPopuli left a comment

Choose a reason for hiding this comment

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

so close :P

cache: pnpm
- run: npm install -g npm@latest # ensure that the globally installed npm is new enough to support OIDC
- run: pnpm install --frozen-lockfile
- name: Publish to NPM
Copy link
Member

Choose a reason for hiding this comment

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

I think you could conditionalize the arguments based on branch name here to enable the different branch releases

Copy link
Member Author

Choose a reason for hiding this comment

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

oh that is a much better idea 👍 that will stop me from having to manage the diff all the time! thanks a million for that

Copy link
Member Author

Choose a reason for hiding this comment

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

I've pushed it in case you want to check, although I think the syntax can really only properly be tested in prod 🫠

Copy link
Member

Choose a reason for hiding this comment

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

I think that should work.

In ember-source I did it as two different steps with a condition https://github.com/emberjs/ember.js/blob/main/.github/workflows/publish-to-npm.yml#L32

@mansona mansona merged commit b1533a2 into master Feb 7, 2026
70 of 71 checks passed
@mansona mansona deleted the release-plan branch February 7, 2026 20:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants