Skip to content

feat: Update editorconfig to use tabs by default - #85

Open
agraven-famedly wants to merge 1 commit into
mainfrom
agraven/tab-indent-size
Open

feat: Update editorconfig to use tabs by default#85
agraven-famedly wants to merge 1 commit into
mainfrom
agraven/tab-indent-size

Conversation

@agraven-famedly

@agraven-famedly agraven-famedly commented Sep 2, 2026

Copy link
Copy Markdown

Update the editorconfig so tabs are the default. The original choices of spaces was not well-motivated and tabs suit us better (we strive to use them everywhere we can for accessibility and flexibility reasons).


Note

Low Risk
Editor and formatter hints only; no runtime or security impact, though editors may suggest tabs for file types that previously defaulted to spaces.

Overview
EditorConfig now treats tabs as the default for all files (indent_style / indent_size on [*]), replacing the previous default of 2 spaces.

Per-language rules are reorganized: explicit 2-space overrides are added for *.dart, *.nix, *.toml, and *.{yaml,yml}. The old tab-only sections for shell, Makefiles, Rust, and Go are removed because they inherit the new global default.

The same content is updated in standards/editorconfig (source) and .editorconfig (generated copy via filegen-activate).

Reviewed by Cursor Bugbot for commit af0a730. Bugbot is set up for automated code reviews on this repo. Configure here.

@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Comment thread standards/editorconfig
@agraven-famedly
agraven-famedly force-pushed the agraven/tab-indent-size branch 2 times, most recently from 0b0ddd4 to 57a2158 Compare September 2, 2026 10:40
@agraven-famedly agraven-famedly changed the title fix: Set indent_size to tab for tab indented files feat: Update editorconfig to use tabs by default Sep 2, 2026
Comment thread standards/editorconfig

@RasmusRendal RasmusRendal left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks cool!
Let's merge it tomorrow if nobody complains before then :)

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 8778b43. Configure here.

Comment thread standards/editorconfig
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants