Skip to content

Commit

Permalink
think this will work around the linux nodejs bug nodejs/node#35889
Browse files Browse the repository at this point in the history
  • Loading branch information
jcolson committed Sep 28, 2021
1 parent 390a44d commit 255c84b
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
4 changes: 3 additions & 1 deletion Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,9 @@ FROM base as test
RUN apt-get install -y git git-lfs python3 python-pip && pip install pre-commit

RUN npm ci
RUN npm run test --verbose
RUN npm run test -- --verbose --testPathIgnorePatterns roll.test.js
# workaround for linux node bug --> https://github.com/nodejs/node/issues/35889
RUN npm run test roll.test.js -- --verbose

# Start me!
# CMD ["node", "index.js"]
Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
"main": "index.js",
"scripts": {
"test": "NODE_OPTIONS=--experimental-vm-modules jest --coverage",
"dockertest": "docker build --target test .",
"dockertest": "docker build --progress plain --target test .",
"preinstall": "git lfs update --force && pre-commit install -t pre-commit -t pre-push",
"preinstall_old": "npx npm-force-resolutions && pre-commit install -t pre-commit -t pre-push"
},
Expand Down

0 comments on commit 255c84b

Please sign in to comment.