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

build: update to node 16 and npm 8 #71

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

Conversation

manbearwiz
Copy link

@manbearwiz manbearwiz commented Jun 30, 2023

I noticed a lot of packages in this repo are very out of date and I wanted to help update them, as I still see this benchmark cited when reviewing new hardware. I think this wll be a good first step to facilitate future updates. I elected to update to node 16 and npm 8 to minimize the number of changes made at once.

I took the following steps.

  • use npm 8 to upgrade lockfile to v2
  • update upath 1.0.4 > 1.2.0 to remove requirement on node: '>=4 <=9'
  • update .nvmrc file to 16
  • update travis build to use node versions that ship with npm 7 and 8 (is travis still setup?)
  • ran install, build, test, and cli with node 16

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