-
Notifications
You must be signed in to change notification settings - Fork 11.9k
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
The engine "pnpm" appears to be invalid. #11214
Comments
Which version of ppm are you using? Might be the same as #11208? |
I use yarn, v1.22.19 in next.js project, node v16.15.0. I don't use pnpm directly |
same problem on codesandbox, doesn't look like a local problem |
Seems you need to use V2 of yarn, its a wont fix in yarn |
How to solve this issue?
I'm also using yarn |
Same problem, I don't use pnpm directly, this "warning chart.js@4.2.1: The engine "pnpm" appears to be invalid." was in my GitLab pipeline. |
Ones who use |
Can you try force version 7.x.x pnpm? https://vercel.com/docs/concepts/deployments/configure-a-build |
This will be fixed in V4.3.0 Workaround is to install the master branch instead of a package version untill it is released |
Tried installing the master branch specifically (in a SvelteKit / Vite app):
Results in this error:
|
You installed the package as a dev dependency, could you try installing in this way: pnpm add https://github.com/chartjs/Chart.js#master |
Thanks. The module export error is persisting, unfortunately. |
I guess that would be necessary as a manual copy-over on a production server as well then? Would not mind an intermediate package release à la 4.2.5 or something… Would be a quick solution for this bug, as the source looks ready for it. @LeeLenaleee |
This issue is fixed from a PR that was done a month ago. Why can't you do a 0.0.1 release just to fix this bug? The issue is just changing the package.json engine from ^7.0.0 to >= 7.0.0. It's really causing a lot of people headaches over something already fixed. |
@ZetiMente sorry about this, I don't have a ton of time to spend on opensource at the moment. I've sent the version bump PR #11257 which would be the final step before a release. |
4.3 is released |
Its only a warning, but you get that warning because you are using yarn V1 and need to use V2 |
Why other modules don't have warning in V1? I can't understand this I don't know if it applies here, Vercel enforces V1 for full support https://vercel.com/guides/does-vercel-support-yarn-2 |
They don't have the warning because they don't use pnpm then, instead they use yarn, npm or something else. Apart from the warning it should work fine. As seen in my comment i linked yarn marked it as a no fix for V1 so you will have to keep the warning or update to yarn 2 |
Upgrading Yarn solved it for me.
|
Issue is fixed in #11209 |
yarnpkg/yarn#7560 (comment) |
Expected behavior
no warning
Current behavior
warning chart.js@4.2.1: The engine "pnpm" appears to be invalid.
Reproducible sample
main
Optional extra steps/info to reproduce
yarn add chart.js
Possible solution
No response
Context
When yarn
Fetching packages...
return warningchart.js version
v4.2.1
Browser name and version
No response
Link to your project
No response
The text was updated successfully, but these errors were encountered: