-
Notifications
You must be signed in to change notification settings - Fork 980
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
Switch to NodeJS 14.15 LTS (fixes #10158) #10159
Conversation
Running this locally results in the following for me:
Because:
I think we should also update the lockfile with this change, the build also outputs:
|
Co-authored-by: Dustin Ingram <di@users.noreply.github.com>
Co-authored-by: Dustin Ingram <di@users.noreply.github.com>
Co-authored-by: Dustin Ingram <di@users.noreply.github.com>
@di did you catch this error on |
I found it by running
I don't see how the |
|
Rebased #10052 on top of this PR to show that the build is fixed. |
Looks like my issues in #10159 (comment) were due to docker-compose not immediately picking up these changes. |
@di I also had some problems with Could you also add |
* Switch to NodeJS 14.15 LTS (fixes pypi#10158) * More finegrained Node version in CI Co-authored-by: Dustin Ingram <di@users.noreply.github.com> * More finegrained Node version in static image Co-authored-by: Dustin Ingram <di@users.noreply.github.com> * More finegrained Node version in web image Co-authored-by: Dustin Ingram <di@users.noreply.github.com> * Use native `npm` version in static Node image Co-authored-by: Dustin Ingram <di@users.noreply.github.com>
Fixes build failure in docker build that started two days ago after
npm 8
released.Uses
npm
versions that is provided by the image, and switches the image to 14.15 which is a minimum required bynpm
8 in case it will be needed.