-
Notifications
You must be signed in to change notification settings - Fork 29
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
fix: specify node version >=14.15 <18
#439
Conversation
kevinsamoei
commented
Jul 11, 2022
•
edited
Loading
edited
- Lighthouse v9 does not work with node v18
- This updates the node version in package.json to match anything between v14.15 and 18
- Updated tests to run on v17.9
>=14.15 <18
netlify.toml
Outdated
@@ -3,7 +3,7 @@ command = "echo 'no op'" | |||
publish = "example" | |||
|
|||
[build.environment] | |||
NODE_VERSION = "16.15.1" | |||
NODE_VERSION = "17.9.1" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's stay on LTS
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we don't have an LTS on v17.*
so I set it to 16.15.1
which is an LTS
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!