mirror of
https://github.com/zed-industries/zed.git
synced 2026-06-01 03:14:56 +07:00
Mention pure style changes in the contributing docs (#39096)
Release Notes: - N/A
This commit is contained in:
parent
12aba6193e
commit
aa14980523
1 changed files with 1 additions and 0 deletions
|
|
@ -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.
|
||||
|
||||
|
|
|
|||
Loading…
Reference in a new issue