Skip to content

Commit 8accfbf

Browse files
mgallienclaude
andcommitted
docs(agents): require repo-wide clang-format and mandatory clang-tidy
Broaden the clang-format requirement beyond ./src to all C++ source files, and add a mandatory run-clang-tidy -fix pass that must report no further changes before a task is considered done. Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com> Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
1 parent 159851c commit 8accfbf

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

AGENTS.md

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -193,7 +193,9 @@ Our C++ code should can make use of C++ 20 standard features whenever possible.
193193

194194
Do not use C++ modules. Use standard header inclusion instead.
195195

196-
After editing or adding C++ source files under `./src`, run `clang-format -i` on the touched files before considering the task done.
196+
After editing or adding any C++ source files, anywhere in the repository, run `clang-format -i` on the touched files before considering the task done.
197+
198+
After editing or adding any C++ source files, anywhere in the repository, run `run-clang-tidy -p build -header-filter='.*' -config-file .clang-tidy -fix` from the repository root and confirm it produces no further changes. This is a mandatory step: apply any fixes the tool makes, review them, re-run the command, and repeat until it reports no changes before considering the task done. Do not skip or silently waive this step; if `run-clang-tidy` cannot be run (for example because `./build` is not configured), state that explicitly rather than proceeding as if it passed.
197199

198200
## macOS Specifics
199201

0 commit comments

Comments
 (0)