Skip to content

Commit

Permalink
ci Keep js deps up-to-date (npm ci)
Browse files Browse the repository at this point in the history
tox will re-install deps if our `deps` change, but not if `package.json`
changes (or, say, the version of Node/npm).
  • Loading branch information
bheesham committed Oct 21, 2024
1 parent 97dafa3 commit 8543624
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 2 deletions.
5 changes: 3 additions & 2 deletions ci/node-install.sh
Original file line number Diff line number Diff line change
Expand Up @@ -3,9 +3,10 @@ set -eu

if [ -f "$TOX_ENV_DIR/bin/node" ]; then
echo Node and friends already installed.
npm ci
exit 0
fi

nodeenv --prebuilt -p --node 18.20.4 "$TOX_ENV_DIR"
npm install -g npm@latest
nodeenv --prebuilt -p --node "$NODE_VERSION" "$TOX_ENV_DIR"
npm install -g "npm@$NPM_VERSION"
npm ci
3 changes: 3 additions & 0 deletions tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,9 @@ skip_missing_interpreters = false
description = base environment
package = wheel
wheel_build_env = .pkg
set_env =
NODE_VERSION=18.20.4
NPM_VERSION=10.9.0
deps =
pytest: -r requirements.txt
eslint,stylelint: nodeenv
Expand Down

0 comments on commit 8543624

Please sign in to comment.