Skip to content

Comments

ci: use npx for npm version instead of global installation#1969

Merged
slesaad merged 1 commit intomainfrom
ci/use-npx
Feb 3, 2026
Merged

ci: use npx for npm version instead of global installation#1969
slesaad merged 1 commit intomainfrom
ci/use-npx

Conversation

@slesaad
Copy link
Member

@slesaad slesaad commented Feb 3, 2026

Description of Changes

Uses npx for npm version for publishing instead of installing globally which we apparently don't have permission for. See this test workflow run

I've already tested this via the test-trusted-publishing branch - this is the workflow run using npm publish --dry-run. It only failed because of the following error:

npm error You cannot publish over the previously published versions: 6.20.3.

But the other steps in the workflow should create a new version.

@netlify
Copy link

netlify bot commented Feb 3, 2026

Deploy Preview for veda-ui ready!

Name Link
🔨 Latest commit 075e4fa
🔍 Latest deploy log https://app.netlify.com/projects/veda-ui/deploys/698217a8c363280008021c7d
😎 Deploy Preview https://deploy-preview-1969--veda-ui.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@slesaad slesaad requested a review from dzole0311 February 3, 2026 15:43
@slesaad
Copy link
Member Author

slesaad commented Feb 3, 2026

@dzole0311 i was waiting to merge the previous PR until I could test that it actually worked, but yesterday GitHub had some major outage and I was just able to run the workflow today to test it and see that it didn't actually worked 😅 So this!

@slesaad slesaad changed the title Use npx for npm version instead of global installation ci: use npx for npm version instead of global installation Feb 3, 2026
@slesaad slesaad requested a review from ifsimicoded February 3, 2026 16:56
Copy link
Contributor

@ifsimicoded ifsimicoded left a comment

Choose a reason for hiding this comment

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

🚀

@slesaad slesaad merged commit 3e69322 into main Feb 3, 2026
10 of 11 checks passed
@slesaad slesaad deleted the ci/use-npx branch February 3, 2026 17:32
@ifsimicoded
Copy link
Contributor

relates to: #1961

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