diff options
| author |  MarkKoz <[email protected]> | 2021-01-12 11:57:10 -0800 |
|---|---|---|
| committer | MarkKoz <[email protected]> | 2021-01-12 11:57:10 -0800 |
| commit | 247eab38e4142918da9aa25555f4f72382faeb2f (patch) | |
| tree | 5f5fb38472fca3d267ccf6eb56338f036c42e6da | |
| parent | Docker: update pipenv (diff) | |
CI: don't do a user install for pre-commit
| -rw-r--r-- | .github/workflows/lint-test-build-push.yaml | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/.github/workflows/lint-test-build-push.yaml b/.github/workflows/lint-test-build-push.yaml index 5aa2180..700fba9 100644 --- a/.github/workflows/lint-test-build-push.yaml +++ b/.github/workflows/lint-test-build-push.yaml @@ -108,11 +108,12 @@ jobs: docker exec snekbox_test /bin/bash -c 'apt-get -y update && apt-get install -y git=1:2.20.*' + # pre-commit's venv doesn't work with user installs. # Skip the flake8 hook because the following step will run it. - name: Run pre-commit hooks run: >- docker exec snekbox_test /bin/bash -c - 'SKIP=flake8 pre-commit run --all-files' + 'PIP_USER=0 SKIP=flake8 pre-commit run --all-files' # This runs `flake8` in the container and asks `flake8` to output # linting errors in the format of the command for registering workflow |