[wp-trac] [WordPress Trac] #28494: Multiple strings in .editorconfig section names should be wrapped in curly brackets

[WordPress Trac]

#28494: Multiple strings in .editorconfig section names should be wrapped in curly
brackets
------------------------------+------------------------------
 Reporter:  netweb            |       Owner:
     Type:  defect (bug)      |      Status:  new
 Priority:  normal            |   Milestone:  Awaiting Review
Component:  Build/Test Tools  |     Version:  trunk
 Severity:  normal            |  Resolution:
 Keywords:  has-patch         |     Focuses:
------------------------------+------------------------------

Comment (by treyhunner):

 Replying to [comment:4 netweb]:
 > Replying to [comment:3 treyhunner]:
 >
 > Thanks for the feedback Trey,
 >
 > > The current syntax is simply wrong, so any change is an improvement.
 The only kind of file [*.json,*.yml] will match is something like
 "word.json,anotherword.yml".
 >
 > It wasn't wrong when we initially added .editorconfig to WordPress in
 February ;)

 Actually it was.  Commas were not meant to denote separate files until we
 decided to add brace expansion in March.  If those globs ever matched
 something without a ".json," in the middle of the filename and a ".yml" at
 the end, it was a bug in the parser being used.  I'm fairly sure none of
 the parsers in February 2013 used commas to denote separate file globs.

 >
 > If we patch it using {curly braces}:
 >
 > * IDE/Text Editors with an up to date implementation of .editorconfig
 0.11.0 spec will match those files.
 >
 > * IDE/Text Editors without an up to date implementation will not match
 the files correctly.
 >
 > Is there a list of common IDE/Text Editors support of the 0.11.0
 .editorconfig spec? Has it been widely implemented?

 This is specific to each editor plugin.  I believe all of the plugins I
 know have been updated to 0.11 for some time (almost all since March
 2013).

--
Ticket URL: <https://core.trac.wordpress.org/ticket/28494#comment:5>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform