Skip to content

Conversation

@monnier
Copy link
Collaborator

@monnier monnier commented Nov 19, 2025

Remove from editorconfig-core.el code that sets tab_width from indent_size and vice-versa, since that's better done (and is done) later in editorconfig.el.

@xuhdev xuhdev requested a review from 10sr November 27, 2025 02:20
@xuhdev
Copy link
Member

xuhdev commented Nov 27, 2025

Test failed:

195/202 Test #195: unset_indent_size_ML .............................***Failed  Required regular expression not found. Regex=[^[
]*indent_size=unset[ 	

]*tab_width=unset[ 	

]*$
]  0.03 sec

indent_size=unset

@monnier monnier force-pushed the core-size-agnostic branch 3 times, most recently from 000182c to a9bf87a Compare November 28, 2025 20:27
* editorconfig-core.el (editorconfig-core-get-properties-hash):
Remove code that sets tab_width from indent_size and vice-versa,
since that's better done (and is done) later in `editorconfig.el`.
Also remove autoload cookie since this file is already
`require`d from the caller anyway.
Remove`confversion` arg, not used.
(editorconfig-core-get-properties): Move to `editorconfig-el`.

* bin/editorconfig-el (editorconfig-core-get-properties):
New function taken from `editorconfig-core.el`.
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