From aa14980523891ee3403975739705e79256989774 Mon Sep 17 00:00:00 2001 From: Kirill Bulatov Date: Mon, 29 Sep 2025 14:02:47 +0300 Subject: [PATCH] Mention pure style changes in the contributing docs (#39096) Release Notes: - N/A --- CONTRIBUTING.md | 1 + 1 file changed, 1 insertion(+) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 8615ba2c9ad324357792772392c49c7899ffa410..9cbac4af2b57f0350fa9f5665e110e0d6e7f6341 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -63,6 +63,7 @@ Although there are few hard and fast rules, typically we don't merge: - New file icons. Zed's default icon theme consists of icons that are hand-designed to fit together in a cohesive manner, please don't submit PRs with off-the-shelf SVGs. - Giant refactorings. - Non-trivial changes with no tests. +- Stylistic code changes that do not alter any app logic. Reducing allocations, removing `.unwrap()`s, fixing typos is great; making code "more readable" — maybe not so much. - Features where (in our subjective opinion) the extra complexity isn't worth it for the number of people who will benefit. - Anything that seems completely AI generated.