stylelint-order 0.6.0 → 0.7.0

package/CHANGELOG.md

@@ -2,9 +2,13 @@
 All notable changes to this project will be documented in this file.
 This project adheres to [Semantic Versioning](http://semver.org/).
 
+## 0.7.0
+
+* Specified `stylelint` in `peerDependencies` rather in `dependencies`. Following [stylelint's plugin guide](https://github.com/stylelint/stylelint/blob/master/docs/developer-guide/plugins.md#peer-dependencies).
+
 ## 0.6.0
 
-* Migrate to `stylelint@8.0.0`.
+* Migrated to `stylelint@8.0.0`.
 
 ## 0.5.0
 * Added autofixing for every rule! Please read docs before using this feature, because each rule has some caveats. stylelint 7.11+ is required for this feature.

package/README.md

@@ -4,8 +4,16 @@ A plugin pack of order related linting rules for [stylelint]. Every rule support
 
 ## Installation
 
+First, install [stylelint]:
+
+```
+npm install stylelint --save-dev
+```
+
+Then install plugin:
+
 ```
-npm install stylelint-order
+npm install stylelint-order --save-dev
 ```
 
 ## Usage

package/index.js

@@ -4,7 +4,7 @@ const createPlugin = require('stylelint').createPlugin;
 const namespace = require('./utils').namespace;
 const rules = require('./rules');
 
-const rulesPlugins = Object.keys(rules).map((ruleName) => {
+const rulesPlugins = Object.keys(rules).map(ruleName => {
 	return createPlugin(namespace(ruleName), rules[ruleName]);
 });
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "stylelint-order",
-  "version": "0.6.0",
+  "version": "0.7.0",
   "description": "A collection of order related linting rules for stylelint.",
   "keywords": [
     "stylelint-plugin",
@@ -28,21 +28,37 @@
   "main": "index.js",
   "dependencies": {
     "lodash": "^4.17.4",
-    "postcss": "^6.0.7",
-    "postcss-sorting": "^3.0.1",
+    "postcss": "^6.0.11",
+    "postcss-sorting": "^3.0.2"
+  },
+  "peerDependencies": {
     "stylelint": "^8.0.0"
   },
   "devDependencies": {
-    "eslint": "^3.19.0",
-    "eslint-config-hudochenkov": "^1.3.0",
-    "jest": "^20.0.4"
+    "eslint": "~4.7.2",
+    "eslint-config-hudochenkov": "~2.0.0",
+    "eslint-config-prettier": "^2.6.0",
+    "eslint-plugin-prettier": "^2.3.1",
+    "husky": "^0.14.3",
+    "jest": "^21.1.0",
+    "lint-staged": "^4.2.2",
+    "prettier": "^1.7.0",
+    "stylelint": "^8.1.1"
   },
   "scripts": {
+    "precommit": "lint-staged",
     "pretest": "eslint .",
     "test": "jest",
     "jest": "jest",
     "watch": "jest --watch",
-    "coverage": "jest --coverage"
+    "coverage": "jest --coverage",
+    "fix": "npm run pretest -- --fix"
+  },
+  "lint-staged": {
+    "*.js": [
+      "npm run fix",
+      "git add"
+    ]
   },
   "jest": {
     "setupFiles": [

package/rules/checkAlphabeticalOrder.js

@@ -4,7 +4,10 @@ module.exports = function checkAlphabeticalOrder(firstPropData, secondPropData)
 	// If unprefixed prop names are the same, compare the prefixed versions
 	if (firstPropData.unprefixedName === secondPropData.unprefixedName) {
 		// If first property has no prefix and second property has prefix
-		if (!postcss.vendor.prefix(firstPropData.name).length && postcss.vendor.prefix(secondPropData.name).length) {
+		if (
+			!postcss.vendor.prefix(firstPropData.name).length &&
+			postcss.vendor.prefix(secondPropData.name).length
+		) {
 			return false;
 		}
 

package/rules/index.js

@@ -1,5 +1,5 @@
 module.exports = {
-	'order': require('./order'),
+	order: require('./order'),
 	'properties-order': require('./properties-order'),
 	'properties-alphabetical-order': require('./properties-alphabetical-order'),
 };

package/rules/order/calcAtRulePatternPriority.js

@@ -30,7 +30,11 @@ module.exports = function calcAtRulePatternPriority(pattern, node) {
 	}
 
 	// doesn't have `name` and `hasBlock`
-	if (!pattern.hasOwnProperty('hasBlock') && !pattern.hasOwnProperty('name') && !pattern.hasOwnProperty('paremeter')) {
+	if (
+		!pattern.hasOwnProperty('hasBlock') &&
+		!pattern.hasOwnProperty('name') &&
+		!pattern.hasOwnProperty('paremeter')
+	) {
 		priority = 1;
 	}
 
@@ -45,7 +49,12 @@ module.exports = function calcAtRulePatternPriority(pattern, node) {
 	}
 
 	// patter has `name`, `parameter`, and `hasBlock`, but it doesn't match all properties
-	if (pattern.hasOwnProperty('name') && pattern.hasOwnProperty('parameter') && pattern.hasOwnProperty('hasBlock') && priority < 30000) {
+	if (
+		pattern.hasOwnProperty('name') &&
+		pattern.hasOwnProperty('parameter') &&
+		pattern.hasOwnProperty('hasBlock') &&
+		priority < 30000
+	) {
 		priority = 0;
 	}
 

package/rules/order/checkOrder.js

@@ -15,14 +15,20 @@ module.exports = function checkOrder(firstNodeData, secondNodeData, allNodesData
 	if (firstNodeIsUnspecified && !secondNodeIsUnspecified) {
 		// If first node is unspecified, look for a specified node before it to
 		// compare to the current node
-		const priorSpecifiedNodeData = _.findLast(allNodesData.slice(0, -1), (d) => Boolean(d.expectedPosition));
+		const priorSpecifiedNodeData = _.findLast(allNodesData.slice(0, -1), d =>
+			Boolean(d.expectedPosition)
+		);
 
 		if (
-			priorSpecifiedNodeData && priorSpecifiedNodeData.expectedPosition
-			&& priorSpecifiedNodeData.expectedPosition > secondNodeData.expectedPosition
+			priorSpecifiedNodeData &&
+			priorSpecifiedNodeData.expectedPosition &&
+			priorSpecifiedNodeData.expectedPosition > secondNodeData.expectedPosition
 		) {
 			stylelint.utils.report({
-				message: sharedInfo.messages.expected(secondNodeData.description, priorSpecifiedNodeData.description),
+				message: sharedInfo.messages.expected(
+					secondNodeData.description,
+					priorSpecifiedNodeData.description
+				),
 				node: secondNodeData.node,
 				result: sharedInfo.result,
 				ruleName: sharedInfo.ruleName,
@@ -45,6 +51,7 @@ module.exports = function checkOrder(firstNodeData, secondNodeData, allNodesData
 	if (unspecified === 'top' && firstNodeIsUnspecified) {
 		return true;
 	}
+
 	if (unspecified === 'top' && secondNodeIsUnspecified) {
 		return false;
 	}
@@ -52,6 +59,7 @@ module.exports = function checkOrder(firstNodeData, secondNodeData, allNodesData
 	if (unspecified === 'bottom' && secondNodeIsUnspecified) {
 		return true;
 	}
+
 	if (unspecified === 'bottom' && firstNodeIsUnspecified) {
 		return false;
 	}

package/rules/order/createExpectedOrder.js

@@ -7,27 +7,17 @@ module.exports = function createExpectedOrder(input) {
 	const order = {};
 	let expectedPosition = 0;
 
-	input.forEach((item) => {
+	input.forEach(item => {
 		expectedPosition += 1;
 
-		if (
-			(
-				_.isString(item)
-				&& item !== 'at-rules'
-				&& item !== 'rules'
-			)
-			|| item === 'less-mixins'
-		) {
+		if ((_.isString(item) && item !== 'at-rules' && item !== 'rules') || item === 'less-mixins') {
 			order[item] = {
 				expectedPosition,
 				description: getDescription(item),
 			};
 		}
 
-		if (
-			item === 'rules'
-			|| item.type === 'rule'
-		) {
+		if (item === 'rules' || item.type === 'rule') {
 			// Convert 'rules' into extended pattern
 			if (item === 'rules') {
 				item = {
@@ -56,10 +46,7 @@ module.exports = function createExpectedOrder(input) {
 			order[item.type].push(nodeData);
 		}
 
-		if (
-			item === 'at-rules'
-			|| item.type === 'at-rule'
-		) {
+		if (item === 'at-rules' || item.type === 'at-rule') {
 			// Convert 'at-rules' into extended pattern
 			if (item === 'at-rules') {
 				item = {

package/rules/order/getDescription.js

@@ -8,7 +8,7 @@ module.exports = function getDescription(item) {
 		'dollar-variables': '$-variable',
 		'at-variables': '@-variable',
 		'less-mixins': 'Less mixin',
-		'declarations': 'declaration',
+		declarations: 'declaration',
 	};
 
 	if (_.isPlainObject(item)) {

package/rules/order/getOrderData.js

@@ -18,11 +18,7 @@ module.exports = function getOrderData(expectedOrder, node) {
 		} else if (utils.isStandardSyntaxProperty(node.prop)) {
 			nodeType = 'declarations';
 		}
-	} else if (
-		node.type === 'rule'
-		&& node.empty
-		&& !node.extend
-	) {
+	} else if (node.type === 'rule' && node.empty && !node.extend) {
 		nodeType = 'less-mixins';
 	} else if (node.type === 'rule') {
 		nodeType = {
@@ -37,7 +33,7 @@ module.exports = function getOrderData(expectedOrder, node) {
 			let prioritizedPattern;
 			let max = 0;
 
-			rules.forEach(function (pattern) {
+			rules.forEach(function(pattern) {
 				const priority = calcRulePatternPriority(pattern, nodeType);
 
 				if (priority > max) {
@@ -72,7 +68,7 @@ module.exports = function getOrderData(expectedOrder, node) {
 			let prioritizedPattern;
 			let max = 0;
 
-			atRules.forEach(function (pattern) {
+			atRules.forEach(function(pattern) {
 				const priority = calcAtRulePatternPriority(pattern, nodeType);
 
 				if (priority > max) {

package/rules/order/index.js

@@ -17,7 +17,7 @@ const messages = stylelint.utils.ruleMessages(ruleName, {
 function rule(expectation, options, context) {
 	context = context || {};
 
-	return function (root, result) {
+	return function(root, result) {
 		const validOptions = stylelint.utils.validateOptions(
 			result,
 			ruleName,

package/rules/order/validatePrimaryOption.js

@@ -10,52 +10,68 @@ module.exports = function validatePrimaryOption(actualOptions) {
 
 	// Every item in the array must be a certain string or an object
 	// with a "type" property
-	if (!actualOptions.every((item) => {
-		if (_.isString(item)) {
-			return _.includes(['custom-properties', 'dollar-variables', 'at-variables', 'declarations', 'rules', 'at-rules', 'less-mixins'], item);
-		}
+	if (
+		!actualOptions.every(item => {
+			if (_.isString(item)) {
+				return _.includes(
+					[
+						'custom-properties',
+						'dollar-variables',
+						'at-variables',
+						'declarations',
+						'rules',
+						'at-rules',
+						'less-mixins',
+					],
+					item
+				);
+			}
 
-		return _.isPlainObject(item) && !_.isUndefined(item.type);
-	})) {
+			return _.isPlainObject(item) && !_.isUndefined(item.type);
+		})
+	) {
 		return false;
 	}
 
 	const objectItems = actualOptions.filter(_.isPlainObject);
 
-	if (!objectItems.every((item) => {
-		let result = true;
-
-		if (item.type !== 'at-rule' && item.type !== 'rule') {
-			return false;
-		}
+	if (
+		!objectItems.every(item => {
+			let result = true;
 
-		if (item.type === 'at-rule') {
-			// if parameter is specified, name should be specified also
-			if (!_.isUndefined(item.parameter) && _.isUndefined(item.name)) {
+			if (item.type !== 'at-rule' && item.type !== 'rule') {
 				return false;
 			}
 
-			if (!_.isUndefined(item.hasBlock)) {
-				result = item.hasBlock === true || item.hasBlock === false;
-			}
+			if (item.type === 'at-rule') {
+				// if parameter is specified, name should be specified also
+				if (!_.isUndefined(item.parameter) && _.isUndefined(item.name)) {
+					return false;
+				}
 
-			if (!_.isUndefined(item.name)) {
-				result = _.isString(item.name) && item.name.length;
-			}
+				if (!_.isUndefined(item.hasBlock)) {
+					result = item.hasBlock === true || item.hasBlock === false;
+				}
+
+				if (!_.isUndefined(item.name)) {
+					result = _.isString(item.name) && item.name.length;
+				}
 
-			if (!_.isUndefined(item.parameter)) {
-				result = (_.isString(item.parameter) && item.parameter.length) || _.isRegExp(item.parameter);
+				if (!_.isUndefined(item.parameter)) {
+					result =
+						(_.isString(item.parameter) && item.parameter.length) || _.isRegExp(item.parameter);
+				}
 			}
-		}
 
-		if (item.type === 'rule') {
-			if (!_.isUndefined(item.selector)) {
-				result = (_.isString(item.selector) && item.selector.length) || _.isRegExp(item.selector);
+			if (item.type === 'rule') {
+				if (!_.isUndefined(item.selector)) {
+					result = (_.isString(item.selector) && item.selector.length) || _.isRegExp(item.selector);
+				}
 			}
-		}
 
-		return result;
-	})) {
+			return result;
+		})
+	) {
 		return false;
 	}
 

package/rules/properties-alphabetical-order/index.js

@@ -16,7 +16,7 @@ const messages = stylelint.utils.ruleMessages(ruleName, {
 function rule(actual, options, context) {
 	context = context || {};
 
-	return function (root, result) {
+	return function(root, result) {
 		const validOptions = stylelint.utils.validateOptions(
 			result,
 			ruleName,

package/rules/properties-order/README.md

@@ -574,18 +574,16 @@ If `order: "flexible"` is using, for sorting it will be treated as `strict`. It
 Given:
 
 ```json
-{
-	"properties-order": [
-		{
-			"order": "flexible",
-			"properties": [
-				"color",
-				"font-size",
-				"font-weight"
-			]
-		}
-	]
-}
+[
+	{
+		"order": "flexible",
+		"properties": [
+			"color",
+			"font-size",
+			"font-weight"
+		]
+	}
+]
 ```
 
 Before:

package/rules/properties-order/addEmptyLineBefore.js

@@ -4,11 +4,11 @@ const _ = require('lodash');
 
 // Add an empty line before a node. Mutates the node.
 module.exports = function addEmptyLineBefore(node, newline) {
-	if (!(/\r?\n/).test(node.raws.before)) {
+	if (!/\r?\n/.test(node.raws.before)) {
 		node.raws.before = _.repeat(newline, 2) + node.raws.before;
-	} else if ((/^\r?\n/).test(node.raws.before)) {
+	} else if (/^\r?\n/.test(node.raws.before)) {
 		node.raws.before = newline + node.raws.before;
-	} else if ((/\r?\n$/).test(node.raws.before)) {
+	} else if (/\r?\n$/.test(node.raws.before)) {
 		node.raws.before = node.raws.before + newline;
 	} else {
 		node.raws.before = node.raws.before.replace(/(\r?\n)/, `${newline}$1`);

package/rules/properties-order/checkEmptyLineBefore.js

@@ -11,17 +11,14 @@ module.exports = function checkEmptyLineBefore(firstPropData, secondPropData, sh
 	const secondPropIsUnspecified = !secondPropData.orderData;
 
 	// Now check newlines between ...
-	const firstPropSeparatedGroup = (!firstPropIsUnspecified)
+	const firstPropSeparatedGroup = !firstPropIsUnspecified
 		? firstPropData.orderData.separatedGroup
 		: sharedInfo.lastKnownSeparatedGroup;
-	const secondPropSeparatedGroup = (!secondPropIsUnspecified)
+	const secondPropSeparatedGroup = !secondPropIsUnspecified
 		? secondPropData.orderData.separatedGroup
 		: sharedInfo.lastKnownSeparatedGroup;
 
-	if (
-		firstPropSeparatedGroup !== secondPropSeparatedGroup
-		&& !secondPropIsUnspecified
-	) {
+	if (firstPropSeparatedGroup !== secondPropSeparatedGroup && !secondPropIsUnspecified) {
 		// Get an array of just the property groups, remove any solo properties
 		const groups = _.reject(sharedInfo.expectation, _.isString);
 

package/rules/properties-order/checkNode.js

@@ -26,10 +26,7 @@ module.exports = function checkNode(node, sharedInfo) {
 		if (child.type === 'decl') {
 			const prop = child.prop;
 
-			if (
-				utils.isStandardSyntaxProperty(prop)
-				&& !utils.isCustomProperty(prop)
-			) {
+			if (utils.isStandardSyntaxProperty(prop) && !utils.isCustomProperty(prop)) {
 				let unprefixedPropName = postcss.vendor.unprefixed(prop);
 
 				// Hack to allow -moz-osx-font-smoothing to be understood
@@ -53,9 +50,9 @@ module.exports = function checkNode(node, sharedInfo) {
 
 		// current node should be a standard declaration
 		if (
-			child.type !== 'decl'
-			|| !utils.isStandardSyntaxProperty(nodeData.node.prop)
-			|| utils.isCustomProperty(nodeData.node.prop)
+			child.type !== 'decl' ||
+			!utils.isStandardSyntaxProperty(nodeData.node.prop) ||
+			utils.isCustomProperty(nodeData.node.prop)
 		) {
 			return;
 		}
@@ -64,10 +61,7 @@ module.exports = function checkNode(node, sharedInfo) {
 		const previousPropData = _.nth(allPropData, -2);
 
 		// Skip first decl
-		if (
-			previousPropData
-			&& (!sharedInfo.context.fix || sharedInfo.disableFix)
-		) {
+		if (previousPropData && (!sharedInfo.context.fix || sharedInfo.disableFix)) {
 			const isCorrectOrder = checkOrder(previousPropData, nodeData, allPropData, sharedInfo);
 
 			if (!isCorrectOrder) {
@@ -85,9 +79,9 @@ module.exports = function checkNode(node, sharedInfo) {
 
 		// if previous node is shared-line comment, use second previous node
 		if (
-			previousNodeData
-			&& previousNodeData.node.type === 'comment'
-			&& previousNodeData.node.raw('before').indexOf('\n') === -1
+			previousNodeData &&
+			previousNodeData.node.type === 'comment' &&
+			previousNodeData.node.raw('before').indexOf('\n') === -1
 		) {
 			previousNodeData = _.nth(allNodesData, -3);
 		}
@@ -103,8 +97,8 @@ module.exports = function checkNode(node, sharedInfo) {
 
 		// previous node should be a standard declaration
 		if (
-			!utils.isStandardSyntaxProperty(previousNodeData.node.prop)
-			|| utils.isCustomProperty(previousNodeData.node.prop)
+			!utils.isStandardSyntaxProperty(previousNodeData.node.prop) ||
+			utils.isCustomProperty(previousNodeData.node.prop)
 		) {
 			return;
 		}

package/rules/properties-order/checkOrder.js

@@ -8,7 +8,10 @@ const checkAlphabeticalOrder = require('../checkAlphabeticalOrder');
 module.exports = function checkOrder(firstPropData, secondPropData, allPropData, sharedInfo) {
 	if (firstPropData.unprefixedName === secondPropData.unprefixedName) {
 		// If first property has no prefix and second property has prefix
-		if (!postcss.vendor.prefix(firstPropData.name).length && postcss.vendor.prefix(secondPropData.name).length) {
+		if (
+			!postcss.vendor.prefix(firstPropData.name).length &&
+			postcss.vendor.prefix(secondPropData.name).length
+		) {
 			return false;
 		}
 
@@ -26,12 +29,12 @@ module.exports = function checkOrder(firstPropData, secondPropData, allPropData,
 	if (firstPropIsUnspecified && !secondPropIsUnspecified) {
 		// If first prop is unspecified, look for a specified prop before it to
 		// compare to the current prop
-		const priorSpecifiedPropData = _.findLast(allPropData.slice(0, -1), (d) => Boolean(d.orderData));
+		const priorSpecifiedPropData = _.findLast(allPropData.slice(0, -1), d => Boolean(d.orderData));
 
 		if (
-			priorSpecifiedPropData
-			&& priorSpecifiedPropData.orderData
-			&& priorSpecifiedPropData.orderData.expectedPosition > secondPropData.orderData.expectedPosition
+			priorSpecifiedPropData &&
+			priorSpecifiedPropData.orderData &&
+			priorSpecifiedPropData.orderData.expectedPosition > secondPropData.orderData.expectedPosition
 		) {
 			stylelint.utils.report({
 				message: sharedInfo.messages.expected(secondPropData.name, priorSpecifiedPropData.name),
@@ -48,25 +51,18 @@ module.exports = function checkOrder(firstPropData, secondPropData, allPropData,
 
 	// Now deal with unspecified props
 	// Starting with bottomAlphabetical as it requires more specific conditionals
-	if (
-		unspecified === 'bottomAlphabetical'
-		&& !firstPropIsUnspecified
-		&& secondPropIsUnspecified
-	) {
+	if (unspecified === 'bottomAlphabetical' && !firstPropIsUnspecified && secondPropIsUnspecified) {
 		return true;
 	}
 
-	if (
-		unspecified === 'bottomAlphabetical'
-		&& secondPropIsUnspecified
-		&& firstPropIsUnspecified
-	) {
+	if (unspecified === 'bottomAlphabetical' && secondPropIsUnspecified && firstPropIsUnspecified) {
 		if (checkAlphabeticalOrder(firstPropData, secondPropData)) {
 			return true;
 		}
 
 		return false;
 	}
+
 	if (unspecified === 'bottomAlphabetical' && firstPropIsUnspecified) {
 		return false;
 	}
@@ -82,6 +78,7 @@ module.exports = function checkOrder(firstPropData, secondPropData, allPropData,
 	if (unspecified === 'top' && firstPropIsUnspecified) {
 		return true;
 	}
+
 	if (unspecified === 'top' && secondPropIsUnspecified) {
 		return false;
 	}
@@ -89,6 +86,7 @@ module.exports = function checkOrder(firstPropData, secondPropData, allPropData,
 	if (unspecified === 'bottom' && secondPropIsUnspecified) {
 		return true;
 	}
+
 	if (unspecified === 'bottom' && firstPropIsUnspecified) {
 		return false;
 	}

package/rules/properties-order/createExpectedOrder.js

@@ -10,7 +10,7 @@ module.exports = function createExpectedOrder(input) {
 	appendGroup(input);
 
 	function appendGroup(items) {
-		items.forEach((item) => appendItem(item, false));
+		items.forEach(item => appendItem(item, false));
 	}
 
 	function appendItem(item, inFlexibleGroup) {
@@ -35,7 +35,7 @@ module.exports = function createExpectedOrder(input) {
 		if (item.order && item.order === 'flexible') {
 			expectedPosition += 1;
 
-			item.properties.forEach((property) => {
+			item.properties.forEach(property => {
 				appendItem(property, true);
 			});
 		} else {

package/rules/properties-order/createFlatOrder.js

@@ -8,7 +8,7 @@ module.exports = function createFlatOrder(order) {
 	appendGroup(order);
 
 	function appendGroup(items) {
-		items.forEach((item) => appendItem(item));
+		items.forEach(item => appendItem(item));
 	}
 
 	function appendItem(item) {

package/rules/properties-order/index.js

@@ -13,14 +13,14 @@ const ruleName = utils.namespace('properties-order');
 
 const messages = stylelint.utils.ruleMessages(ruleName, {
 	expected: (first, second) => `Expected "${first}" to come before "${second}"`,
-	expectedEmptyLineBefore: (property) => `Expected an empty line before property "${property}"`,
-	rejectedEmptyLineBefore: (property) => `Unexpected an empty line before property "${property}"`,
+	expectedEmptyLineBefore: property => `Expected an empty line before property "${property}"`,
+	rejectedEmptyLineBefore: property => `Unexpected an empty line before property "${property}"`,
 });
 
-const rule = function (expectation, options, context) {
+const rule = function(expectation, options, context) {
 	context = context || {};
 
-	return function (root, result) {
+	return function(root, result) {
 		const validOptions = stylelint.utils.validateOptions(
 			result,
 			ruleName,
@@ -31,12 +31,7 @@ const rule = function (expectation, options, context) {
 			{
 				actual: options,
 				possible: {
-					unspecified: [
-						'top',
-						'bottom',
-						'ignore',
-						'bottomAlphabetical',
-					],
+					unspecified: ['top', 'bottom', 'ignore', 'bottomAlphabetical'],
 					disableFix: _.isBoolean,
 				},
 				optional: true,

package/rules/properties-order/validatePrimaryOption.js

@@ -10,37 +10,43 @@ module.exports = function validatePrimaryOption(actualOptions) {
 
 	// Every item in the array must be a string or an object
 	// with a "properties" property
-	if (!actualOptions.every((item) => {
-		if (_.isString(item)) {
-			return true;
-		}
-
-		return _.isPlainObject(item) && !_.isUndefined(item.properties);
-	})) {
+	if (
+		!actualOptions.every(item => {
+			if (_.isString(item)) {
+				return true;
+			}
+
+			return _.isPlainObject(item) && !_.isUndefined(item.properties);
+		})
+	) {
 		return false;
 	}
 
 	const objectItems = actualOptions.filter(_.isPlainObject);
 
 	// Every object-item's "properties" should be an array with no items, or with strings
-	if (!objectItems.every((item) => {
-		if (!Array.isArray(item.properties)) {
-			return false;
-		}
-
-		return item.properties.every((property) => _.isString(property));
-	})) {
+	if (
+		!objectItems.every(item => {
+			if (!Array.isArray(item.properties)) {
+				return false;
+			}
+
+			return item.properties.every(property => _.isString(property));
+		})
+	) {
 		return false;
 	}
 
 	// Every object-item's "emptyLineBefore" must be "always" or "never"
-	if (!objectItems.every((item) => {
-		if (_.isUndefined(item.emptyLineBefore)) {
-			return true;
-		}
-
-		return _.includes(['always', 'never'], item.emptyLineBefore);
-	})) {
+	if (
+		!objectItems.every(item => {
+			if (_.isUndefined(item.emptyLineBefore)) {
+				return true;
+			}
+
+			return _.includes(['always', 'never'], item.emptyLineBefore);
+		})
+	) {
 		return false;
 	}
 

package/utils/isCustomProperty.js

@@ -6,5 +6,5 @@
  */
 
 module.exports = function isCustomProperty(property) {
-	return (property.slice(0, 2) === '--');
+	return property.slice(0, 2) === '--';
 };