Conversation
This branch was successfully deployed
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.
Motivation
Description
pr gateverb with--watch,--interval, and--jsonsupport and aPrGateArgsclap struct wired into the command tree and pull-request routing.src/cli/gate.rs, producingpass | pending | blockedplus categorizedblockersandpendingreasons derived from check runs, review decision, unresolved threads, draft/PR state, mergeable/merge-state, and merge-queue state, and exposingheadOidand cache/review truncation provenance.0for pass,1for blocked, and2for pending (constantGATE_EXIT_PENDINGadded and used by watch mode).src/cli/pr_verbs.rs,src/cli/verbs.rs,src/cli/pull_request.rs, andsrc/cli/mod.rs) and document it indocs/cli/pull-request/gate.mdand the pull-request README.PullRequestGate::evaluateand behaviour rendering, and an end-to-end fake-GitHub regression test that exercises textual output, JSON output, and exit codes; also update the CLI capabilities test expectation to account for the new verb.Testing
cargo fmt --all -- --check, which succeeded.cargo clippy --all-targets --all-features --locked -- -D warnings, which succeeded after addressing warnings.cargo test --all-features --locked) passed (including the new gate unit tests and the added fake-GitHub integration test).python3 scripts/check_comments.pyto validate repository comment rules, which succeeded.make_pr) could not be exercised in this environment due to a missing runtime import in the local MCP server harness; this does not affect the gate implementation or its tests which passed locally.Codex Task