search

editorconfig / editorconfig-jetbrains

A JetBrains IDE plugin supporting the EditorConfig standard
MIT License
291 stars 21 forks source link

Ordering issue #40

Closed ddeboer closed 10 years ago

ddeboer commented 10 years ago

With the following .editorconfig YML files are incorrectly indented with 4 spaces

root = true

[*]
indent_size = 4

[*.yml]
indent_size = 2

While with this config YMLs are correctly indented with 2 spaces:

root = true

[*.yml]
indent_size = 2

[*]
indent_size = 4

So it seems the order matters. I'm not sure whether it should, as I can't find anything about this on the EditorConfig website. I think, however, my first example resembles those on the website most closely and should work as is.

denofevil commented 10 years ago

This issue was caused by new parser incorrectly merging keys in one file and is fixed by https://github.com/editorconfig/editorconfig-core-java/commit/c1c249a8670c004999333f9d5573dfb2e869f110, we will post updated plugin version soon

ddeboer commented 10 years ago

Very cool! Thanks for the great plugin, by the way. :smile: