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

chore: fix pnpm requirements in CONTRIBUTING.md #7134

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

xiaohanyu
Copy link
Contributor

Summary

use pnpm other than 9.x will report an error.

$ pnpm i
 ERR_PNPM_UNSUPPORTED_ENGINE  Unsupported environment (bad pnpm and/or Node.js version)

Your pnpm version is incompatible with "/Users/hanyu/github/logto-io/logto".

Expected version: ^9.0.0
Got: 10.6.2

This is happening because the package's manifest has an engines.pnpm field specified.
To fix this issue, install the required pnpm version globally.

To install the latest version of pnpm, run "pnpm i -g pnpm".
To check your pnpm version, run "pnpm -v".

ref commit:
silverhand-io/essentials@2171b96

Testing

Checklist

  • .changeset
  • unit tests
  • integration tests
  • necessary TSDoc comments

Copy link

COMPARE TO master

Total Size Diff 0 Bytes

Diff by File
Name Diff
.github/CONTRIBUTING.md 0 Bytes

@xiaohanyu xiaohanyu force-pushed the fix-pnpm-requirements branch from 7cb1dc8 to b95f111 Compare March 14, 2025 08:54
use pnpm other than 9.x will report an error.

ref commit:
silverhand-io/essentials@2171b96
@xiaohanyu xiaohanyu force-pushed the fix-pnpm-requirements branch from b95f111 to 1025b4b Compare March 14, 2025 09:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

3 participants