Skip to content

01a07a71 - Omit A38 report status on draft pull requests - #71

Merged
TaprootFreak merged 5 commits into
developfrom
feat/01a07a71-a38-draft-no-status
Sep 7, 2026
Merged

01a07a71 - Omit A38 report status on draft pull requests#71
TaprootFreak merged 5 commits into
developfrom
feat/01a07a71-a38-draft-no-status

Conversation

@TaprootFreakAI

Copy link
Copy Markdown
Collaborator

EN:
Draft pull requests no longer get a red A38 report check. The status appears only after Ready for review, green or red according to the author report.

DE:
Draft-PRs bekommen keinen roten A38-Report-Haken mehr. Der Status erscheint erst nach Ready for review, grün oder rot je nach Author-Report.

Details

On a draft, dfx pr guard publishes no A38 / report (<target>) commit status and exits 0, so the Actions check is not red. Missing evidence is still recorded in the educational comment. Ready for review keeps today's enforce behaviour: valid report → success, missing or invalid report → failure.

Does not fabricate a passing status without evidence.

@TaprootFreakAI
TaprootFreakAI force-pushed the feat/01a07a71-a38-draft-no-status branch from 897800e to d524421 Compare September 7, 2026 15:16
@TaprootFreakAI

Copy link
Copy Markdown
Collaborator Author

EN:
Ready after 2 review passes.
Draft pull requests omit the blocking A38 report status until Ready for review, while the welcome comment still posts.

DE:
Bereit nach 2 Review-Durchläufen.
Draft-PRs lassen den blockierenden A38-Report-Status bis Ready for review weg, der Willkommenskommentar kommt weiter.

Details
  • Head 0bed0f4318295df82a9612eb93870744f52939a9 on develop 1c2fb0e50c47189ead58f8482da63e830ba8e522.
  • Quality and logic: 0 findings on this head. Pass 1 found comment/docs/test gaps; pass 2 is clean after those fixes.
  • Comments: no open review threads.
  • Mergeable: MERGEABLE.
  • Checks at this head: pytest pass.

@TaprootFreakAI
TaprootFreakAI marked this pull request as ready for review September 7, 2026 16:19
@TaprootFreak
TaprootFreak merged commit 425b6ed into develop Sep 7, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants