Skip to content

merge-pr: go ahead without Greptile when it created no review for the head - #1001

Merged
ualtinok merged 1 commit into
mainfrom
land/merge-without-absent-greptile
Oct 27, 2026
Merged

ualtinok merged 1 commit into
mainfrom
land/merge-without-absent-greptile

Conversation

@aft-alfonso

@aft-alfonso aft-alfonso Bot commented Oct 26, 2026

Copy link
Copy Markdown

Greptile stopped reviewing cortexkit/aft PRs this morning (no check run since 06:44Z), and merge-pr.sh refused every PR after its 20-minute wait. Ufuk approved merging without Greptile while it is down.

A head with an existing Greptile review still waits for it, and a failed review still refuses. Only a head where Greptile created nothing within 180 s goes ahead, with a warning line. AFT_MERGE_PR_GREPTILE_ABSENT_GRACE_SECONDS=0 restores the old rule.

… head

Greptile stopped creating reviews on cortexkit/aft (last one 06:44Z on
2026-10-26), and merge-pr refused every PR after a 20-minute wait, so nothing
merged. Ufuk approved merging without Greptile while it is down. A review that
exists is still waited for and a failed one still refuses; only a head Greptile
never touched goes ahead, after 180 s (AFT_MERGE_PR_GREPTILE_ABSENT_GRACE_SECONDS,
0 restores the old rule).

Co-authored-by: alfonso-aft <289616620+alfonso-aft@users.noreply.github.com>
@ualtinok
ualtinok added this pull request to the merge queue Oct 26, 2026
@ualtinok
ualtinok removed this pull request from the merge queue due to a manual request Oct 26, 2026
@ualtinok
ualtinok added this pull request to the merge queue Oct 26, 2026
Merged via the queue into main with commit 21a567d Oct 27, 2026
48 checks passed
@ualtinok
ualtinok deleted the land/merge-without-absent-greptile branch October 27, 2026 08:04
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.

1 participant