Update .editorconfig#2
Conversation
There was a problem hiding this comment.
📋 Dispatch Review — full · 1 file reviewed · effort 🟢 XS
Summary
This PR modifies the repository's .editorconfig configuration file, changing the global trim_trailing_whitespace setting from true to false. It is a single-line change with no accompanying rationale or targeted scope.
Changes
| Layer / File(s) | Summary |
|---|---|
| Editor configuration | |
.editorconfig |
Flips the global trim_trailing_whitespace rule from true to false, disabling automatic trailing-whitespace removal for all file types in editors that honour .editorconfig. |
Review Status
Analyzed · requesting changes · 🟡 1 warning
Reviewed by Upsun Dispatch · 2026-06-18 17:05 UTC
| end_of_line = lf | ||
| insert_final_newline = true | ||
| trim_trailing_whitespace = true | ||
| trim_trailing_whitespace = false |
There was a problem hiding this comment.
🟡 Warning — Global disablement of whitespace trimming silently degrades diff readability and code hygiene across the entire repository with no scoped justification.
Changing trim_trailing_whitespace from true to false disables trailing-whitespace removal globally for every file type in the repository. Editors that respect .editorconfig will silently accumulate whitespace-only lines over time, polluting future diffs with noise and making git blame/git bisect harder to use. If the intent is to preserve trailing spaces only for a specific type (e.g. Markdown, where two trailing spaces encode a hard line-break), the correct fix is a targeted override section rather than a global change:
[*.md]
trim_trailing_whitespace = falseThe global setting should remain true.
No description provided.