Skip to content

fix: remove invalid pnpm config command from workflows#3

Merged
Azdetic merged 9 commits into
mainfrom
fix-pnpm-ci
Jun 29, 2026
Merged

fix: remove invalid pnpm config command from workflows#3
Azdetic merged 9 commits into
mainfrom
fix-pnpm-ci

Conversation

@Azdetic

@Azdetic Azdetic commented Jun 29, 2026

Copy link
Copy Markdown
Owner

No description provided.

Copilot AI review requested due to automatic review settings June 29, 2026 08:24

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 removes an invalid/redundant pnpm config set onlyBuiltDependencies ... step from the CI and release GitHub Actions workflows. The allowed build-script dependencies list is already defined in pnpm-workspace.yaml, so the workflows can rely on repo configuration rather than attempting to set it at runtime.

Changes:

  • Removed the “Approve build scripts” step from .github/workflows/ci.yml.
  • Removed the same step from .github/workflows/release.yml.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
.github/workflows/ci.yml Removes the pnpm config set onlyBuiltDependencies ... step so CI relies on repo config during install.
.github/workflows/release.yml Removes the same invalid/redundant pnpm config step from the release workflow.

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

@Azdetic Azdetic merged commit 657eeb2 into main Jun 29, 2026
1 check 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