You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@camel.apache.org by ap...@apache.org on 2026/07/20 15:00:07 UTC

(camel) branch main updated: Improve AGENTS.md for PR review

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 <ap...@ibm.com>
AuthorDate: Mon Jul 20 12:23:29 2026 +0200

    Improve AGENTS.md for PR review
    
    Signed-off-by: Aurélien Pupier <ap...@ibm.com>
---
 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**.