Skip to content

docs: update Node.js version requirement to >=20 in contributing guide#3545

Merged
ajhollid merged 1 commit intobluewave-labs:developfrom
ZeinebMegaadi:fix/node-version-docs
Apr 20, 2026
Merged

docs: update Node.js version requirement to >=20 in contributing guide#3545
ajhollid merged 1 commit intobluewave-labs:developfrom
ZeinebMegaadi:fix/node-version-docs

Conversation

@ZeinebMegaadi
Copy link
Copy Markdown
Contributor

Describe your changes

Updated the Node.js version requirement in CONTRIBUTING.md from v16+ to v20+ to match the actual requirement specified in server/package.json ("node": ">=20"). This prevents new contributors from setting up with an unsupported Node.js version and hitting cryptic errors.

Write your issue number after "Fixes "

No issue was opened for this change , it's a single-word documentation fix that doesn't require one.

Please ensure all items are checked off before requesting a review. "Checked off" means you need to add an "x" character between brackets so they turn into checkmarks.

  • (Do not skip this or your PR will be closed) I deployed the application locally.
  • (Do not skip this or your PR will be closed) I have performed a self-reviewing and testing of my code.
  • I have included the issue # in the PR.
  • I have added i18n support to visible strings (instead of <div>Add</div>, use):
const { t } = useTranslation();
<div>{t('add')}</div>
  • I have not included any files that are not related to my pull request, including package-lock and package-json if dependencies have not changed
  • I didn't use any hardcoded values (otherwise it will not scale, and will make it difficult to maintain consistency across the application).
  • I made sure font sizes, color choices etc are all referenced from the theme. I don't have any hardcoded dimensions.
  • My PR is granular and targeted to one specific feature.
  • There are two database implementations (MongoDB and PostgreSQL TimescaleDB) and I have taken care of them appropriately.
  • I ran npm run format in server and client directories, which automatically formats your code.
  • I took a screenshot or a video and attached to this PR if there is a UI change.

@ajhollid ajhollid merged commit 33110cf into bluewave-labs:develop Apr 20, 2026
5 checks passed
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.

2 participants