-
-
Notifications
You must be signed in to change notification settings - Fork 954
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
ERR_PNPM_OTHER_PM_EXPECTED This project is configured to use >=pnpm #7940
Comments
The packageManager field only accepts exact versions. This is invalid: |
With version 8.x.x works just fine. I will try putting specific version. |
Because v8 ignored the packageManager field. v9 uses it by the spec. |
is there a chance we can configure this to be less strict? people will often update their package manager (or not) and if they are not using corepack, it won't be done automatically and I don't really want to manually bump the version all the time? Or can it at least work with newer versions? Or only be strict when the patch/major version changes? |
If you want something less strict, then use the engines.pnpm field instead, which accepts ranges. The packageManager field is designed like this by the corepack team. |
And breaking changes can be expected from a major version bump.Maybe worth adding to the release notes when it’s missing?Sent from my iPhoneOn 18 Apr 2024, at 01:26, Zoltan Kochan ***@***.***> wrote:
If you want something less strict, then use the engines.pnpm field instead, which accepts ranges. The packageManager field is designed like this by the corepack team.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>
|
You can disable this check by setting |
thanks! looks like that is not suported by https://github.com/pnpm/action-setup/ but I can file a ticket over there |
I don't get it. If you don't want to use the packageManager field by the spec, why don't you just remove it? I don't see how |
Even if you don't remove it, you can put |
Related PR: #7960 |
I was just saying that it would be cool if it also supported the engines.pnpm field |
Verify latest release
pnpm version
No response
Which area(s) of pnpm are affected? (leave empty if unsure)
No response
Link to the code that reproduces this issue or a replay of the bug
No response
Reproduction steps
I have the following dockerimage:
And in step 8/15 I have the following error:
This was working fine before version 9.
Here is my package.json file:
Describe the Bug
In step 7/15 of docker build start to warn us saying that we need to use pnpm
Expected Behavior
Which Node.js version are you using?
18
Which operating systems have you used?
If your OS is a Linux based, which one it is? (Include the version if relevant)
alpine, fedora
The text was updated successfully, but these errors were encountered: