Skip to content

fix(release): drop --unsafe-perm from publish npm ci - #1681

Merged
lucasmcdonald3 merged 2 commits into
v4.xfrom
fix-v4.x-publish-unsafe-perm
Jul 22, 2026
Merged

fix(release): drop --unsafe-perm from publish npm ci#1681
lucasmcdonald3 merged 2 commits into
v4.xfrom
fix-v4.x-publish-unsafe-perm

Conversation

@lucasmcdonald3

@lucasmcdonald3 lucasmcdonald3 commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Issue #, if available:

Description of changes:

Drop --unsafe-perm from the publish job's npm ci. This flag was removed in npm 12, which we just started using.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Check any applicable:

  • Were any files moved? Moving files changes their URL, which breaks all hyperlinks to the files.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the production release GitHub Actions workflow to align with npm 12+ by removing the deprecated --unsafe-perm flag from the publish job’s dependency install step.

Changes:

  • Remove --unsafe-perm from npm ci in the publish job of the production release workflow.

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

@lucasmcdonald3
lucasmcdonald3 requested a review from Copilot July 22, 2026 21:50
@lucasmcdonald3
lucasmcdonald3 merged commit 4f6e74f into v4.x Jul 22, 2026
34 checks passed
@lucasmcdonald3
lucasmcdonald3 deleted the fix-v4.x-publish-unsafe-perm branch July 22, 2026 21:52

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment thread .github/workflows/prod-release.yml
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