Skip to content
This repository was archived by the owner on Dec 19, 2025. It is now read-only.

Support editorconfig#166

Open
crisff11 wants to merge 2 commits into
segmentio:masterfrom
crisff11:support-editorconfig
Open

Support editorconfig#166
crisff11 wants to merge 2 commits into
segmentio:masterfrom
crisff11:support-editorconfig

Conversation

@crisff11
Copy link
Copy Markdown

Feature: Add support for .editorconfig. Original issue: #138

crisff11 and others added 2 commits May 12, 2025 10:54
Co-authored-by: Jorge Fuertes <jorge1.fuertes@volkswagen-groupservices.com>
Co-authored-by: Carlos Ming <carlos1.ming@volkswagen-groupservices.com>
Co-authored-by: Jorge Fuertes <jorge1.fuertes@volkswagen-groupservices.com>
Co-authored-by: Carlos Ming <carlos1.ming@volkswagen-groupservices.com>
@okuuva
Copy link
Copy Markdown

okuuva commented Jul 1, 2025

Would be extra cool if this supported reading the tab_width and indent_size. From editorconfig home page:

indent_size: a whole number defining the number of columns used for each indentation level and the width of soft tabs (when supported). When set to tab, the value of tab_width (if specified) will be used.
tab_width: a whole number defining the number of columns used to represent a tab character. This defaults to the value of indent_size and doesn't usually need to be specified.

So the priority would be tab_width > indent_size > default.

@heygarrett
Copy link
Copy Markdown

@okuuva I feel like it makes more sense to leave that to the editor, which is presumably already reading from the EditorConfig file.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants