[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