Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Remove packageManager field from package.json #275

Merged
merged 1 commit into from
Nov 23, 2023
Merged

Conversation

MH4GF
Copy link
Owner

@MH4GF MH4GF commented Nov 23, 2023

The following job is failing:

/usr/local/bin/yarn run -s depcruise --output-type mermaid --config .dependency-cruiser.js --focus ^test/bun-workspace/.dependency-cruiser.cjs|^test/bun-workspace/index.js|^test/bun-workspace/packages/foo/index.js|^dist/index.js|^src/installDependencies.ts|^src/options/validateOptions.test.ts|^src/options/validateOptions.ts test/bun-workspace/.dependency-cruiser.cjs test/bun-workspace/index.js test/bun-workspace/packages/foo/index.js dist/index.js src/installDependencies.ts src/options/validateOptions.test.ts src/options/validateOptions.ts
error This project's package.json defines "packageManager": "yarn@pnpm@8.6.5". However the current global version of Yarn is 1.22.21.

Presence of the "packageManager" field indicates that the project is meant to be used with Corepack, a tool included by default with all official Node.js distributions starting from 16.9 and 14.19.
Corepack must currently be enabled by running corepack enable in your terminal. For more information, check out https://yarnpkg.com/corepack.
Error: The process '/usr/local/bin/yarn' failed with exit code 1

https://github.com/MH4GF/dependency-cruiser-report-action/actions/runs/6956926445/job/18950886827?pr=274

Starting with yarn 1.22.21, it appears that the error occurs when another package manager is specified for packageManager.

This field was added for corepack use in local development, but this project is not frequently maintained and should be left unconfigured.

@MH4GF MH4GF self-assigned this Nov 23, 2023
@MH4GF MH4GF added the dev-ex label Nov 23, 2023
@MH4GF MH4GF marked this pull request as ready for review November 23, 2023 00:54
@MH4GF MH4GF merged commit 689bd2c into main Nov 23, 2023
6 checks passed
@MH4GF MH4GF deleted the remove-corepack branch November 23, 2023 00:54
@MH4GF MH4GF mentioned this pull request Nov 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant