chore(greptile): enforce central best practices in reviews - #2115
albertoelias-crossmint wants to merge 9 commits into
Conversation
Co-Authored-By: Alberto Elias <alberto.elias@paella.dev>
|
I'll fix CI failures and address comments from users with write access that start with 'Devin'.
Original prompt from Alberto Elias
|
|
|
No reviewable files after applying ignore patterns. |
…issing PR evidence Co-Authored-By: Alberto Elias <alberto.elias@paella.dev>
Co-Authored-By: Alberto Elias <alberto.elias@paella.dev>
Co-Authored-By: Alberto Elias <alberto.elias@paella.dev>
Co-Authored-By: Alberto Elias <alberto.elias@paella.dev>
Co-Authored-By: Alberto Elias <alberto.elias@paella.dev>
…e/test.md Co-Authored-By: Alberto Elias <alberto.elias@paella.dev>
Co-Authored-By: Alberto Elias <alberto.elias@paella.dev>
Co-Authored-By: Alberto Elias <alberto.elias@paella.dev>
Description
Makes Greptile enforce the central engineering practices in
Paella-Labs/best-practiceson this repo's PRs. These are the same rules Devin and other agents get throughAGENTS.md, and testing is one part of them.context.repos: ["Paella-Labs/best-practices"]: Greptile reads the private repo as cross-repo context through the org's Greptile installation. That repo is consumed here, not reviewed. The rules reference files there instead of copying them, so a change in best-practices changes every review.customContext.files:AGENTS.md, which holds the repo's commands, existing test setup and default PR evidence.manifest.txtplustypescript.md). Each flag cites the file and rule, and only the change itself is in scope, not pre-existing code.code/test.mdand flags (1) no PR evidence of the feature working, (2) unit tests written after the code, (3) new behavior with no committed test, asking for E2E first, and (4) new tests that don't follow the repo's existing setup.Pairs with #2116, which adds the
AGENTS.mdTesting section this config points at; merge them together.Test plan
Config-only.
greptile.jsonparses as JSON. The rules take effect on the next Greptile review.Package updates
None (no package code changed, so no changeset).
Link to Devin session: https://crossmint.devinenterprise.com/sessions/7983bb75c26246418063bfe0d695e3c1
Open in Devin Desktop: https://crossmint.devinenterprise.com/desktop/session/7983bb75c26246418063bfe0d695e3c1?variant=devin
Requested by: @albertoelias-crossmint