Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: stylelint-config-eyeo/index.js

Issue 29643562: Issue 5689 - Add rules for CSS declaration order to stylelint-config-eyeo (Closed) Base URL: https://hg.adblockplus.org/codingtools
Patch Set: Created Dec. 18, 2017, 9:14 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: stylelint-config-eyeo/index.js
===================================================================
--- a/stylelint-config-eyeo/index.js
+++ b/stylelint-config-eyeo/index.js
@@ -14,16 +14,19 @@
* You should have received a copy of the GNU General Public License
* along with Adblock Plus. If not, see <http://www.gnu.org/licenses/>.
*/
"use strict";
module.exports = {
extends: "stylelint-config-recommended",
+ plugins: [
+ "stylelint-order"
+ ],
rules: {
// Opening braces go on their own line
"block-opening-brace-newline-before": "always-multi-line",
"block-opening-brace-newline-after": "always-multi-line",
"block-closing-brace-newline-before": "always-multi-line",
"block-closing-brace-newline-after": "always",
"block-closing-brace-empty-line-before": "never",
@@ -60,11 +63,15 @@
"indentation": 2,
// Line length should be 80 characters or less
"max-line-length": 80,
// Avoid qualifying ID and class names with type selectors
"selector-no-qualifying-type": [true, {
"ignore": ["attribute"]
- }]
+ }],
+
+ // CSS rule declaration order should follow the WordPress CSS
+ // Coding Standards
+ "order/properties-order": require("./css-properties-order")
}
};

Powered by Google App Engine
This is Rietveld