From 6ad207a8b020277cfbcda0dd2c17080a8d4fc1a3 Mon Sep 17 00:00:00 2001 From: Christoph Wurst <1374172+ChristophWurst@users.noreply.github.com> Date: Wed, 13 May 2026 11:35:13 +0200 Subject: [PATCH] docs(agents): document PR review workflow with fixup commits AI-assisted: Claude Code (claude-sonnet-4-6) Signed-off-by: Christoph Wurst <1374172+ChristophWurst@users.noreply.github.com> --- AGENTS.md | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/AGENTS.md b/AGENTS.md index 3932e36426..31b5aed5c8 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -111,6 +111,17 @@ After completing code changes: - There is a [contributing doc](./.github/CONTRIBUTING.md) with suggestions 8. The user will review and commit when ready +### PR Review Workflow + +Once a branch is pushed and under review, **do not force-push**. Reviewers track changes incrementally — a force-push destroys that history and forces them to re-read the full diff from scratch. + +Instead, address feedback with **fixup commits**: +```bash +git commit --fixup= # targets the specific commit being corrected +``` + +The branch will be rebased and squashed into a clean history before merge (CI enforces this). The failing "clean history" CI check is intentional and expected during review — ignore it until the PR has a positive review, then rebase to clean up. + ### Commit Message Format All commits must include two lines at the end: