Skip to content

Setup NPM trusted publishing#57

Merged
andrewjensen merged 2 commits into
mainfrom
adj/npm-trusted-publishing
Jun 23, 2026
Merged

Setup NPM trusted publishing#57
andrewjensen merged 2 commits into
mainfrom
adj/npm-trusted-publishing

Conversation

@andrewjensen

@andrewjensen andrewjensen commented Jun 22, 2026

Copy link
Copy Markdown
Collaborator

This changes our publish workflow configuration so we can use NPM trusted publishing. I've already made the config changes on the NPM side, so once this merges, the publish should happen.


Rovo Dev code review: Rovo Dev is reviewing this pull request…
Refresh the page in a few minutes to see the results.

Comment thread .github/workflows/publish.yaml Outdated

- run: yarn publish
- name: Publish to npm
run: npm publish --provenance

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ Maintainability - DevOps

Since @get-dx/backstage-plugin is a scoped package, npm defaults to restricted access — add --access public to avoid a publish failure: npm publish --provenance --access public.

Details

📖 Explanation: Since this is a scoped package (@get-dx/backstage-plugin), npm defaults to restricted (private) access. Without --access public, the publish will fail unless the package was explicitly configured as public on npm already.

Uses AI. Verify results. Give Feedback

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Added

@andrewjensen andrewjensen merged commit 951a4b1 into main Jun 23, 2026
4 checks 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