01a0855d - Require contents write for Guard Ready mutations - #92
Conversation
|
EN: DE: DetailsHead Comment gate: this wrap-up only. No review threads. HTTP 200 with unchanged Codex quality noted a missing period on the tip commit subject (pre-existing on the previously ready #92; rebase kept it). Codex logic noted falsey empty |
markPullRequestReadyForReview with GITHUB_TOKEN needs contents: write. Without it GraphQL returns HTTP 200 and leaves isDraft unchanged.
d3e9094 to
5443b8f
Compare
EN:
dfx pr guard's GITHUB_TOKEN must have
contents: writesomarkPullRequestReadyForReviewcan leave draft. HTTP 200 with unchangedisDraftis now an explicit error.DE:
Der GITHUB_TOKEN von dfx pr guard braucht
contents: write, damitmarkPullRequestReadyForReviewden Draft aufhebt. HTTP 200 ohne Änderung anisDraftist jetzt ein expliziter Fehler.Details
Fork PR DFXswiss/backend#5465: Guard decided Ready, then
_transitionraisedPR lifecycle mutation failed (HTTP 200). GraphQL with the author token setisDraft: false; the Actions token did not. The documented GitHub requirement iscontents: write.