Closed bravo-kernel closed 2 years ago
Should .nvmrc
be updated in this PR also? 🙂
Darn, that is a very, very good catch sir. Totally missed that due to not having .nvmrc
in my reference repo 🤦.
Can be safely merged now + the env var is no longer needed and can be deleted .
This change requires Netflify environment value for
NODE_VERSION
to be updated from14
to16
:You can safely change the env var (before merging this PR) without breaking the live site. Optionally, you can re-trigger the preview-deploy for this PR within Netlify before merging.
DO NOT MERGE WITHOUT UPDATING THE ENV VAR FIRST !!!