Skip to content

Commit c35be7a

Browse files
dschogitster
authored andcommitted
ci: run expensive tests on push builds to integration branches
Derrick Stolee suggested [1] that expensive tests should be run at a regular cadence rather than on every PR iteration. Gate GIT_TEST_LONG on push builds to the integration branches (next, master, main, maint) so that the EXPENSIVE prereq is satisfied there but not during PR validation, where the extra minutes of wall-clock time do not justify themselves. [1] https://lore.kernel.org/git/e1e8837f-7374-4079-ba87-ab95dd156e33@gmail.com/ Helped-by: Derrick Stolee <derrickstolee@github.com> Assisted-by: Claude Opus 4.6 Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de> Signed-off-by: Junio C Hamano <gitster@pobox.com>
1 parent f190135 commit c35be7a

1 file changed

Lines changed: 9 additions & 0 deletions

File tree

ci/lib.sh

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -314,6 +314,15 @@ export DEFAULT_TEST_TARGET=prove
314314
export GIT_TEST_CLONE_2GB=true
315315
export SKIP_DASHED_BUILT_INS=YesPlease
316316

317+
# Enable expensive tests on push builds to integration branches, but
318+
# not on PR builds where the extra time is not justified for every
319+
# iteration.
320+
case "$GITHUB_EVENT_NAME,$CI_BRANCH" in
321+
push,*next*|push,*master*|push,*main*|push,*maint*)
322+
export GIT_TEST_LONG=YesPlease
323+
;;
324+
esac
325+
317326
case "$distro" in
318327
ubuntu-*)
319328
# Python 2 is end of life, and Ubuntu 23.04 and newer don't actually

0 commit comments

Comments
 (0)