Skip to content

Commit d88d727

Browse files
peffgitster
authored andcommitted
ci: drop linux-clang job
Since the linux-asan-ubsan job runs using clang under Linux, there is not much point in running a separate clang job. Any errors that a normal clang compile-and-test cycle would find are likely to be a subset of what the sanitizer job will find. Since this job takes ~14 minutes to run in CI, this shaves off some of our CPU load (though it does not affect end-to-end runtime, since it's typically run in parallel and is not the longest job). Technically this provides us with slightly less signal for a given run, since you won't immediately know if a failure in the sanitizer job is from using clang or from the sanitizers themselves. But it's generally obvious from the logs, and anyway your next step would be to fix the probvlem and re-run CI, since we expect all of these jobs to pass normally. Signed-off-by: Jeff King <[email protected]> Signed-off-by: Junio C Hamano <[email protected]>
1 parent ec69152 commit d88d727

File tree

1 file changed

+0
-3
lines changed

1 file changed

+0
-3
lines changed

.github/workflows/main.yml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -246,9 +246,6 @@ jobs:
246246
fail-fast: false
247247
matrix:
248248
vector:
249-
- jobname: linux-clang
250-
cc: clang
251-
pool: ubuntu-latest
252249
- jobname: linux-sha256
253250
cc: clang
254251
pool: ubuntu-latest

0 commit comments

Comments
 (0)