Skip to content

fix: add .editorconfig for consistent code style#3786

Merged
ImprovedTube merged 3 commits intocode-charity:masterfrom
karthikeyansundaram2:fix/issue-3785-sidebar-no-page-margin-option-still-leav
Apr 16, 2026
Merged

fix: add .editorconfig for consistent code style#3786
ImprovedTube merged 3 commits intocode-charity:masterfrom
karthikeyansundaram2:fix/issue-3785-sidebar-no-page-margin-option-still-leav

Conversation

@karthikeyansundaram2
Copy link
Copy Markdown
Contributor

@karthikeyansundaram2 karthikeyansundaram2 commented Apr 14, 2026

Summary

Relates to #3785

Adds a file to enforce consistent code style across editors:

  • 2-space indentation
  • LF line endings
  • UTF-8 charset
  • Trim trailing whitespace
  • Insert final newline

This helps contributors maintain consistent formatting which reduces diff noise and makes the codebase easier to maintain.


Contribution by @karthikeyansundaram2

…ll leaving a 2-inch m

Implements the changes requested in issue code-charity#3785.

Closes code-charity#3785
@ImprovedTube
Copy link
Copy Markdown
Member

hi and thanks! @karthikeyansundaram2
nice, can you adjust it to the format we use?

@karthikeyansundaram2
Copy link
Copy Markdown
Contributor Author

Updated! Switched to tab indentation to match the project's existing code style. Let me know if anything else needs adjusting.

@karthikeyansundaram2
Copy link
Copy Markdown
Contributor Author

karthikeyansundaram2 commented Apr 14, 2026 via email

@karthikeyansundaram2
Copy link
Copy Markdown
Contributor Author

Updated! Simplified the .editorconfig to match your codebase style:

  • indent_style = tab with indent_size = 1 (matching your existing JS/JSON files)
  • Removed the [*.md] override section
  • Kept lf, utf-8, trim trailing whitespace, final newline

Let me know if you'd like any other adjustments! 🙌

@ImprovedTube ImprovedTube merged commit 15c67ce into code-charity:master Apr 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants