Skip to content

Commit 27ef16b

Browse files
Update REVIEW.md to cover the impact of changes upon plugins
1 parent 3805e5f commit 27ef16b

1 file changed

Lines changed: 5 additions & 1 deletion

File tree

‎REVIEW.md‎

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -167,7 +167,11 @@ Features that aren't fully stabilized ship behind the **experimental feature fla
167167
- **Gate not-yet-stable work behind the flag, off by default.** A user who hasn't opted in never sees in-progress behavior.
168168
- **It's an early-access opt-in, not a kill switch.** The flag is for users who want features early — not a lever for us to disable things in the field.
169169

170-
## 13. PR hygiene
170+
## 13. Consider impact on plugins
171+
172+
Plugins might conflict with each other. New core features might cause plugins to break even if they don't change the plugin API. The review process must consider the impact of the current chance on plugins in the wild.
173+
174+
## 14. PR hygiene
171175

172176
- Focused and reviewable: aim for the **~500 LOC / 10-file** ceiling; split larger work into stacked PRs.
173177
- Title/branch follow `ADFA-####`; description says *what changed, why, how it was verified*, and flags anything intentionally out of scope (e.g. "UI-only, no unit test").

0 commit comments

Comments
 (0)