Skip to content

update npm version to minimum version required for OIDC publish#887

Merged
Ryang-21 merged 1 commit intomasterfrom
update-publish-workflow
Feb 27, 2026
Merged

update npm version to minimum version required for OIDC publish#887
Ryang-21 merged 1 commit intomasterfrom
update-publish-workflow

Conversation

@Ryang-21
Copy link
Contributor

No description provided.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the npm publishing GitHub Actions workflow to ensure the npm CLI is new enough to support OIDC/trusted publishing during release publishes.

Changes:

  • Add a workflow step to upgrade npm before installing dependencies and publishing.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

registry-url: 'https://registry.npmjs.org'

- name: Update npm for trusted publishing
run: npm install -g npm@11.5.1
Copy link

Copilot AI Feb 27, 2026

Choose a reason for hiding this comment

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

The workflow pins npm to an exact patch version (11.5.1) but the PR title suggests you only need a minimum version for OIDC trusted publishing. Consider either (a) documenting why this exact version is required (e.g., a linked upstream requirement) or (b) using a semver range (like the required major/minor) so the workflow can pick up patch security fixes automatically.

Suggested change
run: npm install -g npm@11.5.1
run: npm install -g npm@^11.5.1

Copilot uses AI. Check for mistakes.
@github-actions
Copy link

Size Change: 0 B

Total Size: 3.52 MB

ℹ️ View Unchanged
Filename Size
dist/stellar-base.js 2.6 MB
dist/stellar-base.min.js 920 kB

compressed-size-action

@Ryang-21 Ryang-21 merged commit 2ffce47 into master Feb 27, 2026
12 checks passed
@Ryang-21 Ryang-21 deleted the update-publish-workflow branch February 27, 2026 17:55
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.

3 participants