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

Update node version in CONTRIBUTING.md #3715

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

Conversation

scape76
Copy link

@scape76 scape76 commented Dec 8, 2024

Contributing.md had an outdated version of node, that misses APIs for using cpy package in drizzle-seed build

Fixes #3707

Contributing.md had an outdated version of node, that misses APIs for using cpy package in drizzle-seed build
@huuyafwww
Copy link

@scape76
Thanks for fixing it.
I was having the same problem as well.
Slightly unrelated to the original issue, but if possible, how about adding an additional fix to match the pnpm install version to the packageManager listed in package.json?
I think it would be a more helpful contribution guide if you specify the version of pnpm.

before fix.

# https://pnpm.io/installation
npm install -g pnpm

after fix.

# https://pnpm.io/installation
npm install -g pnpm@9.7.0

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.

[BUG]: TypeError: os.availableParallelism is not a function
2 participants