[wp-trac] [WordPress Trac] #29792: Grunt: Add a precommit task to check for CSS syntax errors

[WordPress Trac]

#29792: Grunt: Add a precommit task to check for CSS syntax errors
---------------------------------+------------------
 Reporter:  helen                |       Owner:
     Type:  enhancement          |      Status:  new
 Priority:  normal               |   Milestone:  4.2
Component:  Build/Test Tools     |     Version:
 Severity:  normal               |  Resolution:
 Keywords:  has-patch 4.2-early  |     Focuses:
---------------------------------+------------------

Comment (by lkraav):

 Replying to [comment:7 GaryJ]:
 > CSSComb does have a `-l` or `--lint` option, but it's not very helpful
 at the moment, not reporting lines and columns etc. CSSLint would be
 favourable over CSSComb just for error checking of specific non-standards
 issues.

 Yes, this is https://github.com/csscomb/csscomb.js/issues/245

 I did extensive research on csslint vs csscomb yesterday for my own
 company. I found csscomb to be significantly further along for being able
 to define an exact detailed coding standard configuration and then having
 the tool not only lint to it, but automatically beautify your source and
 it actually coming out nearly 100% right. Yandex guys have built csscomb
 on [https://github.com/tonyganch/gonzales-pe/blob/master/doc/AST-
 Description.md gonzales-pe, an AST generator] which looks like a building
 block sitting at the same level where [https://github.com/CSSLint/parser-
 lib/ parser-lib] is sitting for csslint.

 Both are very good tools and might very likely coexist in the build chain,
 but both also have a number of outstanding bugs that could really use
 attention. If there was a crowdfund for getting work done on them, I'd
 contribute in an instant. csscomb is the current choice for my team setup
 here, but as I'm understanding both a lot better now, I will be
 contributing to both and re-evaluating periodically. Wasn't able to find
 anything else at the same maturity level.

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