Skip to content

Commit

Permalink
CI: Don't run poetry update (#1586)
Browse files Browse the repository at this point in the history
This causes `poetry` to regenerate the `poetry.lock` file with newer versions
of dependencies, which defeats the point of checking in the lock file in the
first place.

This mirrors a similar change applied to SAW's CI in
GaloisInc/saw-script@67a25a8.
The use of `poetry update` isn't actively causing problems with Cryptol's CI
yet, but better to be safe than sorry.
  • Loading branch information
RyanGlScott authored Nov 3, 2023
1 parent 7295fc9 commit d602858
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 2 deletions.
1 change: 0 additions & 1 deletion cryptol-remote-api/run_rpc_tests.sh
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,6 @@ get_server() {
}

echo "Setting up python environment for remote server clients..."
poetry update
poetry install

echo "Typechecking code with mypy..."
Expand Down
1 change: 0 additions & 1 deletion cryptol-remote-api/test_docker.sh
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,6 @@ pushd $DIR/python
NUM_FAILS=0

echo "Setting up python environment for remote server clients..."
poetry update
poetry install

export CRYPTOL_SERVER_URL="$PROTO://localhost:8080/"
Expand Down

0 comments on commit d602858

Please sign in to comment.