You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Jan 25, 2023. It is now read-only.
We should print a log message in the build logs when this happens. The log message should warn but have no colors.
We should only do so when the site uses a custom Node.js version, as opposed to using the default Node.js version. The default Node.js is currently 12.18.0 but could differ depending on when the repository was linked, so we should not use the Node.js version value to determine whether the Node.js version has been customized.
The text was updated successfully, but these errors were encountered:
Background
Sites can specify a custom Node.js version by using a
.nvmrc
file,.node_version
file orNODE_VERSION
environment variable.Problem
Specifying the minor/patch version number for Node.js is a bad pattern since:
According to the following list of the Node.js versions used in the last 7 days, per build, this would impact 18% of builds.
Solution
We should print a log message in the build logs when this happens. The log message should warn but have no colors.
We should only do so when the site uses a custom Node.js version, as opposed to using the default Node.js version. The default Node.js is currently
12.18.0
but could differ depending on when the repository was linked, so we should not use the Node.js version value to determine whether the Node.js version has been customized.The text was updated successfully, but these errors were encountered: