Skip to content

Conversation

matijs
Copy link
Member

@matijs matijs commented Sep 19, 2025

Because we want to prevent accidental use of npm, we have added ignore-scripts=true to .npmrc. The presence of .npmrc means that pnpm will also write configuration options to this file instead of to pnpm-workspace.yaml.

Because of this, configuration options were moved back to .npmrc from pnpm-workspace.yaml.

As a bonus, this allows us to document the settings.

Because we want to prevent accidental use of *npm*, we have added
`ignore-scripts=true` to .npmrc. The presence of .npmrc means that pnpm
will also write configuration options to this file instead of to
pnpm-workspace.yaml.

Because of this, configuration options were moved back to .npmrc from
pnpm-workspace.yaml.

As a bonus, this allows us to document the settings.
@matijs matijs requested review from Robbert and Yolijn September 19, 2025 14:58
@matijs matijs requested a review from a team as a code owner September 19, 2025 14:58
Copy link

vercel bot commented Sep 19, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
example Ready Ready Preview Comment Sep 19, 2025 2:58pm

Copy link

Copy link
Member

@Robbert Robbert left a comment

Choose a reason for hiding this comment

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

And as a bonus, more things still work (mostly) with npm

Great move 💯

@Robbert Robbert merged commit 3c9928a into main Sep 19, 2025
8 checks passed
@Robbert Robbert deleted the chore/npmrc branch September 19, 2025 15:06
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