Skip to content

Conversation

@maxstevens-nl
Copy link
Collaborator

@maxstevens-nl maxstevens-nl commented Dec 10, 2025

@antfu/eslint-config added some rules for pnpm-workspaces. Only excluding catalogs was not enough, we have to exclude them from the rule pnpm/yaml-enforce-settings.

Copy link
Collaborator

@Gerbuuun Gerbuuun left a comment

Choose a reason for hiding this comment

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

Not really sure what these things inside pnpm-workspace do... Also, should the pnpm lock be updated like this?

@maxstevens-nl
Copy link
Collaborator Author

maxstevens-nl commented Dec 10, 2025

@Gerbuuun

Not really sure what these things inside pnpm-workspace do...

  • trustPolicy: 'no-downgrade' means that we don't install packages where the trust-level is downgraded (docs)
  • shellEmulator: true allows you to specify scripts in package.json as if you were running on a linux-based system cross-platform.

Also, should the pnpm lock be updated like this?

Yeah let's keep that in the renovate PR's.

@maxstevens-nl maxstevens-nl changed the title deps: update @antfu/eslint-config chore: prep for changes in @antfu/eslint-config Dec 10, 2025
@maxstevens-nl maxstevens-nl merged commit 6b47873 into main Dec 10, 2025
7 checks passed
@maxstevens-nl maxstevens-nl deleted the deps/antfu-eslint-config branch December 10, 2025 10:11
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