This is an automated email from the ASF dual-hosted git repository.
apupier pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git
The following commit(s) were added to refs/heads/main by this push:
new 2c450448ef46 Improve AGENTS.md for PR review
2c450448ef46 is described below
commit 2c450448ef4632b2543ea8de1b6108d4c3336b64
Author: Aurélien Pupier <[email protected]>
AuthorDate: Mon Jul 20 12:23:29 2026 +0200
Improve AGENTS.md for PR review
Signed-off-by: Aurélien Pupier <[email protected]>
---
AGENTS.md | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/AGENTS.md b/AGENTS.md
index 30cc545d8f1c..f2f62990d95c 100644
--- a/AGENTS.md
+++ b/AGENTS.md
@@ -71,6 +71,14 @@ marked ready for review.
- Request review from **at least 2 relevant committers** using `gh pr edit
--add-reviewer`.
- When all comments on the Pull Request are addressed (by providing a fix or
providing more explanation) and the PR checks are green, re-request review on
existing reviewers so that they are aware that the new changeset is ready to be
reviewed.
+### Doing a review
+
+When an AI agent is doing a review:
+
+- Wait until PR checks are green as they will already catch most trivial
issues using less resources
+- It must challenge the code and ensure that it respects all conventions
+- For Dependabot PRs, either do not review them or be able to do a real
review: check for deprecated APIs, removed features, or breaking changes in the
changelog
+
### Merge Requirements
- An agent MUST NOT merge a PR if there are any **unresolved review
conversations**.
