Skip to content

Commit cab174a

Browse files
kim-emclaude
andcommitted
perf(ci): skip crossref dump on non-PR builds
The dump only feeds the crossref-tags-bridge artifact, which is itself only emitted on pull_request_target events (see the existing emit guard). Skip the dump too — saves a couple seconds per push to master and bors/staging branches. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
1 parent e2d8975 commit cab174a

1 file changed

Lines changed: 4 additions & 2 deletions

File tree

.github/workflows/build_template.yml

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -748,8 +748,10 @@ jobs:
748748
- name: dump cross-reference tags to TSV
749749
# Walks Mathlib.CrossRef.tagExt and writes one TSV row per @[stacks ...] /
750750
# @[kerodon ...] / @[wikidata ...] tag for the privileged workflow_run
751-
# job (crossref_review.yml) to consume. The script has no network access
752-
# and enforces a 2 MB output cap.
751+
# job (crossref_review.yml) to consume. No network, defence-in-depth
752+
# size cap (the trusted bound is in mathlib-ci's post-comment.sh).
753+
# Only useful when there's a PR to comment on, so skip on push runs.
754+
if: github.event.pull_request.number != ''
753755
run: lake env lean --run scripts/dump_crossref_tags.lean crossref-tags.tsv
754756

755757
- name: prepare crossref bridge outputs

0 commit comments

Comments
 (0)