sindresorhus / atom-editorconfig

Helps developers maintain consistent coding styles between different editors
https://atom.io/packages/editorconfig
MIT License
812 stars 80 forks source link

Case sensitive; editorconfig spec says it should not be? #209

Closed mhulse closed 6 years ago

mhulse commented 6 years ago

Atom 1.23.1 Atom EditorConfig 2.2.2

Involved .editorconfig-files

root = true

[*]
charset = utf-8
end_of_line = lf
insert_final_newline = true
trim_trailing_whitespace = false
indent_size = 4
indent_style = tab

[vagrantfile]
indent_size = 2
indent_style = space

Directory structure

.editorconfig
Vagrantfile

Noticing that vagrantfile does not match the file name of Vagrantfile, though, the docs say:

Currently all properties and values are case-insensitive. They are lowercased when parsed.

Thanks!

florianb commented 6 years ago

The cited sentence applies for properties and their values, not sections. You're welcome.

mhulse commented 6 years ago

Ahhh, you are right! Sorry that I did not catch that myself.

In fact, there's no reference to the case sensitivity of sections. I'll follow up with EditorConfig repo/folks.

Thanks again!

florianb commented 6 years ago

You're welcome - i wasn't aware this could be an issue. Happy holidays! 🎄