Skip to content

Commit 1ca6394

Browse files
authored
Use node LTS in Alpine (#2621)
Using 13 led to a problem when they updated to 13.7.0, which breaks our ES module test suite integration. It's not clear to me if those are bugs or not. One change is we now get file://X paths instead of X in node-esm-loader.js, but working around that isn't enough, as even once the path is correct, it doesn't even try to load the env.js file, it just says it doesn't have the requested export, so the behavior has changed quite a bit it seems, perhaps intentionally. Rather than spend more time, it seems safer to just use the LTS release there. (But if this is intended and it reaches LTS, we'll need to investigate more.) fixes #2615
1 parent b880950 commit 1ca6394

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

.travis.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -122,7 +122,7 @@ jobs:
122122
compiler: gcc
123123
env: ARCH=x86_64
124124
before_install:
125-
- docker run -w /src -dit --name alpine -v $(pwd):/src node:13-alpine
125+
- docker run -w /src -dit --name alpine -v $(pwd):/src node:lts-alpine
126126
- alpine() { docker exec -it alpine "$@"; }
127127
install:
128128
- alpine apk update

0 commit comments

Comments
 (0)