chore: dismiss Dependabot review requests#266
Conversation
🤖 Codex PR Review✅ Merge allowed: 2 finding(s) reported but none are blocking ℹ️ Other Findings1. 🟡 [MEDIUM] Reliability in
|
27f9c44 to
0b08242
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 0b0824267e
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
c94db90 to
52b99ad
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 52b99adde7
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
52b99ad to
78d65d5
Compare
78d65d5 to
dbfb10d
Compare
Co-Authored-By: Codex <noreply@openai.com>
dbfb10d to
3de4825
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 3de4825807
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| latest_request_actor="$(gh api \ | ||
| "repos/${GITHUB_REPOSITORY}/issues/${PR_NUMBER}/timeline?per_page=100" \ | ||
| --jq '[.[] | select(.event == "review_requested" and .requested_reviewer.login == "Pigbibi")][-1].actor.login // ""')" |
There was a problem hiding this comment.
Paginate timeline events before choosing the latest request
For a Dependabot PR with more than 100 timeline events, this only inspects the first API page, so [-1] need not be the latest Pigbibi request. If Dependabot made the original request in that page and a maintainer re-requested review later, the job identifies the old Dependabot actor and deletes the currently pending manual request, defeating the preservation check. Fetch all pages (or otherwise query the newest matching event) before deciding to remove it.
Useful? React with 👍 / 👎.
What changed
Why
These repositories already auto-merge eligible non-major Dependabot updates after CI, but CODEOWNERS creates redundant review notifications. Event-actor and timeline checks let the workflow remove only bot-generated requests without touching human review requests, including race conditions while a run is queued.
Validation