Skip to content

Commit c0ebf19

Browse files
committed
build: Disable even temporarily running 'pre-commit run --all-files' at the start of CI already
1 parent f72a95e commit c0ebf19

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

.github/workflows/ci.yaml

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -96,12 +96,13 @@ jobs:
9696
# NB: Repeated below in push-container-image: job; must keep in sync!
9797
java-version: "21"
9898

99+
# TODO Explore further how to appropriately use "env -i" isolation to see failure locally (due to missing protoc, at this stage, while still with AFDS, before Flox)
99100
# TODO Remove this duplication; normally this only runs at the end, only here to debug:
100101
# This must ALWAYS run AFTER actions/setup-java, due to https://github.com/google/google-java-format/pull/1228/files.
101-
- name: pre-commit run --all-files
102-
uses: flox/activate-action@v1
103-
with:
104-
command: tools/protoc/protoc.bash && pre-commit run --all-files
102+
# - name: pre-commit run --all-files
103+
# uses: flox/activate-action@v1
104+
# with:
105+
# command: tools/protoc/protoc.bash && pre-commit run --all-files
105106

106107
- name: Install graphviz
107108
# NB: We unfortunately cannot cache this, see https://github.com/enola-dev/enola/issues/823

0 commit comments

Comments
 (0)