Multiple strings in .editorconfig section names should be wrapped in curly brackets

[treyhunner]

The following sections are invalid:

• *.json,*.yml
• *.txt,wp-config-sample.php

They should instead be:

• {*.json,*.yml}
• {*.txt,wp-config-sample.php}

See ​http://editorconfig.org/#file-format-details

This is also a problem in bbpress: ​https://bbpress.trac.wordpress.org/changeset/5383

[netweb]

Trey, thanks, I presume still wrapped in square brackets? eg [{*.json,*.yml}] per patch ​2614.diff?

Presumably then many of the projects via ​editorconfig/wiki/Projects-Using-EditorConfig are 'doing it wrong'

A quick look at some of the 'big' projects the only one I saw using braces was Mozilla ​here

[treyhunner]

Yes that is correct. Square brackets for the section and curly braces to denote the either-or conditions in the shell glob.

I have only noticed one project "doing it wrong" in this way before. I'll try to comb through some of the other large projects I'm aware of that use EditorConfig.

If you find/found any others that do it incorrectly without braces, please let me know and I'll file an issue with them. Thanks!

[netweb]

Created tickets and patches for ​#WordPress28494 and ​#BuddyPress5699

[netweb]

In 5385:

Multiple strings in .editorconfig sections should be wrapped in curly brackets
Props treyhunner. Fixes #2614