serratus-bio / serratus.io

Front-end code for Serratus project website
https://serratus.io
GNU Affero General Public License v3.0
11 stars 11 forks source link

Use node version 16 #195

Closed victorlin closed 1 year ago

victorlin commented 2 years ago

(preview)

Summary of changes

¹ https://github.blog/2021-02-02-npm-7-is-now-generally-available/#changes-to-the-lockfile

I chose to regenerate package-lock.json (remove and re-run npm install) because I was getting weird errors with the recommended approach of running npm install to update the existing lockfile:

npm ERR! node-pre-gyp ERR! install response status 404 Not Found on https://github.com/Automattic/node-canvas/releases/download/v2.8.0/canvas-v2.8.0-node-v93-darwin-unknown-arm64.tar.gz 
…
npm ERR! Package pangocairo was not found in the pkg-config search path.
npm ERR! Perhaps you should add the directory containing `pangocairo.pc'
npm ERR! to the PKG_CONFIG_PATH environment variable
npm ERR! No package 'pangocairo' found
npm ERR! gyp: Call to 'pkg-config pangocairo --libs' returned exit status 1 while in binding.gyp. while trying to load binding.gyp
npm ERR! gyp ERR! configure error 
npm ERR! gyp ERR! stack Error: `gyp` failed with exit code: 1

I think regenerating is fine as long as we test the preview build thoroughly to make sure everything still works as expected.

Related issues

Checklist

victorlin commented 1 year ago

I clicked around the preview and all the pages from the toolkit seem to be on par with the currently deployed site. Going to merge this now.