Skip to content

Commit 33d031b

Browse files
authored
Remove unneeded clang-format CI job (#345)
### What We don't need this because `pre-commit` runs `clang-format`. It's also failing because of a widespread issue with GitHub CI related to the apt update of Ubuntu 22.04: https://github.com/orgs/community/discussions/120966 Which you can see in my CI job here: https://github.com/TileDB-Inc/TileDB-Vector-Search/actions/runs/8815965169/job/24200539999?pr=343 So in this PR we just remove it to fix the CI pipeline.
1 parent 17513cd commit 33d031b

File tree

1 file changed

+0
-12
lines changed

1 file changed

+0
-12
lines changed

.github/workflows/check-formatting.yml

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -17,18 +17,6 @@ on:
1717
- "**/.md"
1818

1919
jobs:
20-
# TODO(paris): pre-commit also runs clang-format. Leaving for now to gain confidence that it
21-
# is doing the same thing as running clang-format directly, but remove this job once that's been
22-
# confirmed
23-
check-formatting:
24-
runs-on: ubuntu-latest
25-
steps:
26-
- uses: actions/checkout@v3
27-
- name: "Check clang-format"
28-
run: |
29-
source $GITHUB_WORKSPACE/scripts/ci/check_clang_format.sh
30-
shell: bash
31-
3220
# Based on: https://github.com/TileDB-Inc/TileDB-Cloud-Py/blob/main/.github/workflows/tiledb-cloud-py.yaml#L15
3321
run-pre-format:
3422
runs-on: ubuntu-latest

0 commit comments

Comments
 (0)