Add ignoreFunctions: [] to length-zero-no-unit

lib/rules/length-zero-no-unit/__tests__/index.js

@@ -620,3 +620,14 @@ testRule({
 		},
 	],
 });
+
+testRule({
+	ruleName,
+	config: [true, { ignoreFunctions: ['var', '--custom'] }],
+	accept: [
+		{
+			code: '.foo { margin-top: var(--y, 0px); margin-bottom: --custom(0px); };',
+			description: 'Should ignore especified functions',
+		},
+	],
+});

lib/rules/length-zero-no-unit/index.js

@@ -16,16 +16,29 @@ const ruleMessages = require('../../utils/ruleMessages');
 const styleSearch = require('style-search');
 const validateOptions = require('../../utils/validateOptions');
 const valueParser = require('postcss-value-parser');
+//const { truncate } = require('lodash');
 
 const ruleName = 'length-zero-no-unit';
 
 const messages = ruleMessages(ruleName, {
 	rejected: 'Unexpected unit',
 });
 
-function rule(actual, secondary, context) {
+function rule(actual, options, context) {
 	return (root, result) => {
-		const validOptions = validateOptions(result, ruleName, { actual });
+		const validOptions = validateOptions(
+			result,
+			ruleName,
+			{ actual },
+			{
+				actual: options,
+				possible: {
+					ignore: ['custom-properties'],
+					ignoreFunctions: [_.isString],
+				},
+				optional: true,
+			},
+		);
 
 		if (!validOptions) {
 			return;
@@ -83,13 +96,36 @@ function rule(actual, secondary, context) {
 		});
 
 		function check(value, node, ignorableIndexes = []) {
-			if (optionsMatches(secondary, 'ignore', 'custom-properties') && isCustomProperty(value)) {
+			if (optionsMatches(options, 'ignore', 'custom-properties') && isCustomProperty(value)) {
 				return;
 			}
 
+			// This was another option although I didn't know how to make it work
+			// const parsedValue = valueParser(value);
+			// parsedValue.walk((valueNode) => {
+			// 	const functionName = valueNode.value;
+			// 	if (optionsMatches(options, 'ignoreFunctions', valueNode.value)) {
+			// 		console.log(valueNode.value);
+			// 		return;
+			// 	}
+			// });
+
 			const fixPositions = [];
 
 			styleSearch({ source: value, target: '0' }, (match) => {
+				const ignoreFunctions = (options && options.ignoreFunctions) || [];
+
+				if (
+					ignoreFunctions.length > 0 &&
+					ignoreFunctions.map((functionName) => {
+						if (value.includes(functionName)) return true;
+
+						return false;
+					})
+				) {
+					return;
+				}
+
 				const index = match.startIndex;
 
 				// Given a 0 somewhere in the full property value (not in a string, thanks