Force npm + node versions for UI#1173
Conversation
This Fixes a few things in the build. - First it forces a specific version for npm + node - the LTS version - Second, it allows cargo to leverage docker in the event your local NPM fails (this may need to be configured better) - Finally, this provides a better caching mechanism for cargo so we're not constantly farming out to `npm`. This is pretty critical for the docker fallback.
|
@jerbly Could really use your help trying this out. If I've done this right, then you should have nothing to change because you're on LTS of node. Additionally, any machine that's on a different version will fallback to using PTAL and let me know what you think! |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1173 +/- ##
=======================================
- Coverage 80.0% 79.9% -0.1%
=======================================
Files 108 109 +1
Lines 8440 8447 +7
=======================================
- Hits 6757 6755 -2
- Misses 1683 1692 +9 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
@jsuereth - To have |
…sion-and-docker-backup
This Fixes a few things in the build.
npm. This is pretty critical for the docker fallback.