Introducing stylelint to lint and alert css styling errors (#6356)

Automattic · aa78bab39a248ba5df045a7c056a42f8fb94e519 · sendilkumarn committed with nylen Jul 6, 2016 · Jul 6, 2016 · aa78bab
1 parent 7473eba
commit aa78bab39a248ba5df045a7c056a42f8fb94e519
Unified Split

Showing with 62 additions and 1 deletion.

+59 −0 .stylelintrc

+3 −1 package.json
diff --git a/.stylelintrc b/.stylelintrc
@@ -0,0 +1,59 @@
+{
+  "rules": {
+    "color-hex-case": "lower",
+    "color-no-invalid-hex": true,
+
+    "function-calc-no-unspaced-operator": true,
+    "function-comma-space-after": "always-single-line",
+    "function-comma-space-before": "never",
+    "function-name-case": "lower",
+    "function-url-quotes": "always",
+    "function-whitespace-after": "always",
+
+    "number-leading-zero": "always",
+    "number-no-trailing-zeros": true,
+    "length-zero-no-unit": true,
+
+    "string-no-newline": true,
+    "string-quotes": "single",
+
+    "unit-case": "lower",
+    "unit-no-unknown": true,
+    "unit-whitelist": ["px", "%", "deg", "ms", "em", "vh", "vw", "rem"],
+
+    "value-list-comma-space-after": "always-single-line",
+    "value-list-comma-space-before": "never",
+
+    "shorthand-property-no-redundant-values": true,
+
+    "property-case": "lower",
+
+    "declaration-block-no-duplicate-properties": true,
+    "declaration-block-no-ignored-properties": true,
+    "declaration-block-trailing-semicolon": "always",
+    "declaration-block-single-line-max-declarations": 0,
+    "declaration-block-semicolon-space-before": "never",
+    "declaration-block-semicolon-space-after": "always-single-line",
+    "declaration-block-semicolon-newline-before": "never-multi-line",
+    "declaration-block-semicolon-newline-after": "always-multi-line",
+
+    "block-closing-brace-newline-after": "always",
+    "block-closing-brace-newline-before": "always-multi-line",
+    "block-no-empty": true,
+    "block-opening-brace-newline-after": "always-multi-line",
+    "block-opening-brace-space-before": "always",
+
+    "selector-attribute-brackets-space-inside": "never",
+    "selector-attribute-operator-space-after": "never",
+    "selector-attribute-operator-space-before": "never",
+    "selector-combinator-space-after": "always",
+    "selector-combinator-space-before": "always",
+    "selector-pseudo-class-case": "lower",
+    "selector-pseudo-class-parentheses-space-inside": "always",
+    "selector-pseudo-element-case": "lower",
+    "selector-pseudo-element-colon-notation": "double",
+    "selector-pseudo-element-no-unknown": true,
+    "selector-type-case": "lower",
+    "selector-no-id": true
+  }
+}
diff --git a/package.json b/package.json
@@ -139,7 +139,8 @@
     "test-client": "NODE_ENV=test NODE_PATH=test:client TEST_ROOT=client test/runner.js",
     "test-server": "NODE_ENV=test NODE_PATH=test:server:client TEST_ROOT=server test/runner.js",
     "test-test": "NODE_ENV=test NODE_PATH=test:client TEST_ROOT=test test/runner.js",
-    "lint": "bin/run-lint"
+    "lint": "bin/run-lint",
+    "css-lint": "stylelint 'client/**/*.scss' --syntax scss"
   },
   "devDependencies": {
     "babel-eslint": "6.0.4",
@@ -171,6 +172,7 @@
     "sinon": "1.12.2",
     "sinon-chai": "2.7.0",
     "socket.io": "1.3.7",
+    "stylelint": "^6.5.1",
     "supertest": "^1.1.0",
     "webpack-dev-server": "1.11.0"
   }