Repository navigation
ci: skip the heavy unit-tests suite for docs-only merge groups - #16413
Merged
Merged
Conversation
The docs-only skip only looked at pull_request events, so every merge group ran the full two-platform suite even when nothing but docs/ had changed. The changes job now also diffs the merge group's base against its speculative merge commit and skips the heavy jobs when the whole union is docs-only. Parser doc pages under docs/content/supported_tools/parsers/ no longer count as docs-only on either event, because unittests/test_parsers.py checks them against dojo/tools/. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Maffooch
enabled auto-merge
October 19, 2026 15:57
auto-merge was automatically disabled
October 19, 2026 16:07
Pull request was closed
Maffooch
enabled auto-merge
October 19, 2026 16:38
blakeaowens
approved these changes
Oct 19, 2026
devGregA
approved these changes
Oct 19, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Shortcut: sc-16697
Summary
The docs-only skip in
unit-tests.yml(#15699, #15712) only looked atpull_requestevents. Onmerge_groupthe detection step never ran, so every queued docs PR rebuilt both platforms and ran the full suite before landing.This extends the
changesjob to the merge queue:merge_group, it diffsmerge_group.base_shaagainstmerge_group.head_sha. The head commit includes every entry queued ahead, so the heavy jobs skip only when that whole union touches nothing outsidedocs/. The code under test is then identical to the base, which already passed through the queue.docs/content/supported_tools/parsers/) no longer count as docs-only on either event.unittests/test_parsers.pychecks that every parser has a doc page and every doc page has a parser, so renaming or deleting one can break the suite.docs_onlystaysfalseand the full suite runs.Unit Tests Completeaccepts a skipped dependency only whendocs_onlyistrue.I ran the step's script locally against real
devcommits: a docs-only commit (#16406) givestrue, a code fix (#16409) givesfalse, and bogus SHAs givefalsewith exit 0.actionlintis clean. The realmerge_grouppath can only be exercised once this is ondev.🤖 Generated with Claude Code