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

WordPress Trac noreply at wordpress.org
Tue Jun 24 06:18:13 UTC 2014


#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):

 Yes the curly brace syntax was added in 0.11.0 which was released in March
 2013.

 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".

 If you really don't want to use the curly brace notation you can always
 expand each glob into separate sections like jQuery still does in theirs:
 https://github.com/jquery/jquery/blob/949dfe52bc228efe2d0cb2d696b822ef2a5723d1/.editorconfig

 Keep in mind that the curly brace notation is very common now and jQuery
 was one of the first projects to adopt the notation which is why they
 still use the old notation.

 So in summary: I vote for the curly brace notation, but manually expanding
 into separate sections would work just fine as well.  I'll just be glad to
 see the notation fixed to something parseable.

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


More information about the wp-trac mailing list