inveniosoftware / invenio-cli

CLI module for Invenio
https://invenio-cli.readthedocs.io
MIT License
10 stars 43 forks source link

Fix check of development requirements #341

Closed wgresshoff closed 1 year ago

wgresshoff commented 1 year ago

:heart: Thank you for your contribution!

Description

Checking of development requirements with (documented) Node.js V16 installed leads to

Pre requisites not met.
Errors: Node wrong version.Got [16, 17, 1] expected 14

Same problem with npm. This PR upgrades the check to the versions mentioned in the docs.

Checklist

Ticks in all boxes and 🟢 on all GitHub actions status checks are required to merge:

Third-party code

If you've added third-party code (copy/pasted or new dependencies), please reach out to an architect.

Reminder

By using GitHub, you have already agreed to the GitHub’s Terms of Service including that:

  1. You license your contribution under the same terms as the current repository’s license.
  2. You agree that you have the right to license your contribution under the current repository’s license.