Skip to content

ci(greptile): scope auto-reviews to maintainer only#1063

Merged
carlos-alm merged 2 commits into
mainfrom
ci/greptile-include-authors
May 4, 2026
Merged

ci(greptile): scope auto-reviews to maintainer only#1063
carlos-alm merged 2 commits into
mainfrom
ci/greptile-include-authors

Conversation

@carlos-alm
Copy link
Copy Markdown
Contributor

Summary

  • Adds .greptile/config.json with an includeAuthors allowlist so Greptile only auto-reviews PRs from carlos-alm.
  • Other contributors (and bots like Dependabot) won't trigger automatic reviews; they can still request one on demand by commenting @greptileai.

Test plan

  • Confirm Greptile does not post a review on PRs opened by other accounts after this lands on main.
  • Confirm @greptileai manual trigger still works on any PR.

Adds .greptile/config.json with an includeAuthors allowlist so Greptile
only triggers automatic reviews on PRs from the maintainer. Other
contributors can still request a review on demand by commenting
@greptileai on their PR.
@claude
Copy link
Copy Markdown

claude Bot commented May 4, 2026

Claude finished @carlos-alm's task —— View job


PR Review: ci(greptile): scope auto-reviews to maintainer only

Todo List

  • Read and analyze the changed file
  • Phase 1: Root Cause Analysis
  • Phase 2: Configuration Change Detection
  • Phase 3: Test Integrity Check
  • Phase 4: Code Quality Assessment
  • Phase 5: Backlog Compliance Evaluation
  • Provide final assessment with rating

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented May 4, 2026

Greptile Summary

Adds .greptile/config.json with an includeAuthors allowlist to restrict Greptile auto-reviews to carlos-alm, preventing unsolicited reviews on Dependabot and other contributor PRs. The change is a minimal, correctly-formed JSON config with no code logic involved.

Confidence Score: 5/5

Safe to merge — single config file addition with no code or logic changes.

The change is one small JSON file with a well-understood Greptile configuration key. No logic, no security implications, no risk of regression.

No files require special attention.

Important Files Changed

Filename Overview
.greptile/config.json New Greptile config that restricts auto-reviews to carlos-alm via the includeAuthors allowlist; valid JSON, correct schema.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[PR opened / updated] --> B{Author in\nincludeAuthors?}
    B -- Yes: carlos-alm --> C[Greptile auto-review triggered]
    B -- No: any other author --> D[No auto-review]
    D --> E{Commenter calls\n@greptileai?}
    E -- Yes --> C
    E -- No --> F[PR proceeds without Greptile review]
Loading

Reviews (1): Last reviewed commit: "ci(greptile): only auto-review PRs autho..." | Re-trigger Greptile

@carlos-alm carlos-alm merged commit 0dd948f into main May 4, 2026
13 checks passed
@carlos-alm carlos-alm deleted the ci/greptile-include-authors branch May 4, 2026 22:05
@github-actions github-actions Bot locked and limited conversation to collaborators May 4, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant