Skip to content

Update .editorconfig #2

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
shawnawsu wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update .editorconfig #2

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .editorconfig
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,4 +9,4 @@ indent_size = 4
indent_style = space
end_of_line = lf
insert_final_newline = true
trim_trailing_whitespace = true
trim_trailing_whitespace = false

@upsun-dispatch-dev upsun-dispatch-dev Bot Jun 18, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 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 = false

The global setting should remain true.