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

Bump Node version in package.json, re-generate package-lock.json #820

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

Conversation

tidoust
Copy link
Collaborator

@tidoust tidoust commented Oct 14, 2024

The package-lock.json file was generated with a now obsolete version of npm and any attempt to run npm install with a more recent version of npm starts with converting the file to the new JSON format. This update does this so that installing the code (or in a job) does not require creating changes to a repo file.

The package.json file also referenced a now obsolete and no longer supported version of Node.js (v14). Using the opportunity to bump the versions to something more current (>v20.14 for Node, >v10.7 for npm). Any sufficiently recent install of Node.js will match these constraints, and this will allow using more advanced Node.js features (including fetch to replace the dependency on request down the road).

@tobie, I do not know whether this can have any consequence on the server deployment. For the auto-update script, I'm going to create a PR right after this one to switch to a GitHub action, and that action actually needs the new version of package-lock.json (otherwise it's going to update it itself...). I let you check for the web server.

The `package-lock.json` file was generated with a now obsolete version of npm
and any attempt to run `npm install` with a more recent version of npm starts
with converting the file to the new JSON format. This update does this so
that installing the code (or in a job) does not require creating changes to a
repo file.

The `package.json` file also referenced a now obsolete and no longer supported
version of Node.js (v14). Using the opportunity to bump the versions to
something more current (>v20.14 for Node, >v10.7 for npm). Any sufficiently
recent install of Node.js will match these constraints, and this will allow
using more advanced Node.js features (including `fetch` to replace the
dependency on `request` down the road).
@tidoust tidoust requested a review from tobie October 14, 2024 12:37
@tidoust tidoust changed the title Bump Node version in package.json, re-genereate package-lock.json Bump Node version in package.json, re-generate package-lock.json Oct 14, 2024
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.

1 participant