Skip to content

Commit ced6e0c

Browse files
bootc-bot[bot]bootc-dev Bot
andauthored
Sync common files from infra repository (#41)
Synchronized from bootc-dev/infra@ff66c1c. Signed-off-by: bootc-dev Bot <bot@bootc.dev> Co-authored-by: bootc-dev Bot <bot@bootc.dev>
1 parent ab87e3c commit ced6e0c

3 files changed

Lines changed: 16 additions & 1 deletion

File tree

.bootc-dev-infra-commit.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
fb9e91b161adacd1c0e9144a76b64159c482b92e
1+
ff66c1c724faeb83b5cd2af36d9a5d2129ecf95a

.cursorrules

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
AGENTS.md

REVIEW.md

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -139,6 +139,20 @@ Generally, just restate the commit message.
139139

140140
Where it makes sense, it is OK to include additional details though.
141141

142+
### Further changes on top of existing commits
143+
144+
If you have followup fixes (whether that's part of a local loop or
145+
as part of addressing PR review), it is generally encouraged to *squash*
146+
the fixes into the prior commit. Do not create generically-named "Update <file>" commits
147+
or "Address review feedback" or "Fix cargo fmt" commits.
148+
149+
This applies equally when an AI tool (e.g. Gemini, Copilot) suggests a
150+
change via a review comment — applying the suggestion creates a new commit
151+
with an auto-generated subject. That commit should be squashed before the
152+
PR is merged.
153+
154+
In other words either a commit "stands alone" with its own rationale or it doesn't.
155+
142156
### Keeping PRs Current
143157

144158
Keep PRs rebased on main. When CI failures are fixed in other PRs, rebase to

0 commit comments

Comments
 (0)