[wp-trac] [WordPress Trac] #26905: Follow CSS coding standards for SASS/SCSS files

[WordPress Trac]

#26905: Follow CSS coding standards for SASS/SCSS files
------------------------------+------------------------------
 Reporter:  morganestes       |       Owner:
     Type:  defect (bug)      |      Status:  new
 Priority:  normal            |   Milestone:  Awaiting Review
Component:  Build/Test Tools  |     Version:  3.8
 Severity:  normal            |  Resolution:
 Keywords:                    |     Focuses:  ui
------------------------------+------------------------------
Changes (by netweb):

 * keywords:  has-patch 2nd-opinion =>
 * component:  Administration => Build/Test Tools


Comment:

 Rather than patching our files, we should add a SCSS Lint Grunt task

 In 26905.diff:

 * Adds a new Grunt task `grunt scsslint` to lint .scss files in src/bp-
 templates/bp-legacy/css/
 * Adds the `scsslint` task to WP's grunt `precommit` task to validate
 files before commit

 https://www.npmjs.com/package/grunt-scss-lint
 https://github.com/brigade/scss-lint

 The rules in the patch in `.scss-lint.yml` get us pretty close to matching
 WordPress Core CSS standards, a few tweaks to either the .scss files in
 core or to these rules should get us where we'd like to be.

 https://make.wordpress.org/core/handbook/coding-standards/css/

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