Skip to content

Commit 3ae11db

Browse files
committed
feat(reviewer): do not tag maintainer after first successful review
1 parent ef1c6db commit 3ae11db

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,7 @@ The reviewer maintains one welcome/status comment and the following labels:
7171
- `human-reviewable`
7272
- `ai-review-failed`
7373

74-
It mentions `GITHUB_REVIEW_MAINTAINER` once per commit when the automated review transitions to approved.
74+
It mentions `GITHUB_REVIEW_MAINTAINER` only the first time a pull request transitions to approved. Later commits are reviewed normally without repeatedly notifying the maintainer.
7575

7676
### AI-assisted issue triage
7777

src/reviews/lifecycle.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -237,7 +237,7 @@ async function setLifecycleStatus(
237237
if (
238238
input.status === "ready" &&
239239
input.headSha &&
240-
state.lastNotifiedSha !== input.headSha
240+
state.lastNotifiedSha === null
241241
) {
242242
const maintainer = process.env.GITHUB_REVIEW_MAINTAINER?.replace(/^@/, "");
243243
if (maintainer)

0 commit comments

Comments
 (0)