Merge guidance omits required AOT and Engine checks
Maintainers usually reply within 1 day
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 62/100
- Issue type
- Documentation
- Clarity
- Clearly specified
- Activity status
- Active
- Tech stack
- csharp, git, github
- Domain
- documentation
Research direction
The mismatch lives in .agents/skills/issue-pr-loop/references/repository.md, in the Merge gate section, which lists two required checks for main. Compare that list against the live branch protection output from gh api repos/thomhurst/TUnit/branches/main/protection, which the issue says requires eight checks across Ubuntu, Windows and macOS. Done means the guide matches the live configuration or points readers to the API as the source of truth, with no change to the merge wrapper's validation.
Written by the indexing model from the issue text.
Description
Summary
The issue/PR loop's repository guidance lists only two required CI checks for main, while GitHub currently requires eight. Update the guidance so agents assessing merges have an accurate account of the branch protection.
Reproduction
Read .agents/skills/issue-pr-loop/references/repository.md, Merge gate, on main at d8badacb13. Compare its two-check list against gh api repos/thomhurst/TUnit/branches/main/protection.
Expected and actual
The guide should describe the current required checks, or point to the live protection API as authoritative. The API currently requires modularpipeline on Ubuntu/macOS, Engine tests on Ubuntu/Windows/macOS, and AOT on Ubuntu/Windows/macOS; the guide lists only modularpipeline on Ubuntu/macOS. The merge wrapper still checks all checks, so this documentation mismatch did not bypass validation for #7406.
Please align the merge guidance with the live protection configuration.
- Dominant language
- C#
- Stars
- 4k
- Forks
- 139
- Avg merge
- 9h 6m
- Merged PRs (30d)
- 282
Getting set up
Starts the project's dev container in your browser, under your own GitHub account.
- No Dockerfile or Docker Compose file
- Has a pull request template
- Read the contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from thomhurst/TUnit
-
bug P3
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
Maintainers usually reply within 1 day
-
bug P3
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
Maintainers usually reply within 1 day
-
bug P3
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
Maintainers usually reply within 1 day
-
bug P2
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
Maintainers usually reply within 1 day
-
bug P3
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
Maintainers usually reply within 1 day
Similar issues
-
area/resource-registry status/triage
Difficulty 1/5 1-3 hours Newbie friendliness 76/100
Altinn/altinn-auth#4821 ·
Maintainers usually reply within 1 day
-
a11y community:contribution
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
microsoft/fluentui-blazor#5590 · 1 comment ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 79/100
JetBrains/resharper-unity#2510 ·
-
Difficulty 1/5 1-3 hours Newbie friendliness 78/100
QuantConnect/Documentation#2838 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
AvaloniaUI/Avalonia#22697 · 1 comment ·
Maintainers usually reply within 1 day