stylelint-scss - npm Package Compare versions

Comparing version 3.17.0 to 3.17.1

CHANGELOG.md

@@ -0,1 +1,5 @@
+# 3.17.1
+
+- Fixed: `function-color-relative` false positives for CSS `filter` property.
+
 # 3.17.0
@@ -2,0 +6,0 @@

dist/rules/at-mixin-named-arguments/index.js

@@ -13,10 +13,14 @@ "use strict";
 
-function _toConsumableArray(arr) { return _arrayWithoutHoles(arr) || _iterableToArray(arr) || _nonIterableSpread(); }
+function _toConsumableArray(arr) { return _arrayWithoutHoles(arr) || _iterableToArray(arr) || _unsupportedIterableToArray(arr) || _nonIterableSpread(); }
 
-function _nonIterableSpread() { throw new TypeError("Invalid attempt to spread non-iterable instance"); }
+function _nonIterableSpread() { throw new TypeError("Invalid attempt to spread non-iterable instance.\nIn order to be iterable, non-array objects must have a [Symbol.iterator]() method."); }
 
-function _iterableToArray(iter) { if (Symbol.iterator in Object(iter) || Object.prototype.toString.call(iter) === "[object Arguments]") return Array.from(iter); }
+function _unsupportedIterableToArray(o, minLen) { if (!o) return; if (typeof o === "string") return _arrayLikeToArray(o, minLen); var n = Object.prototype.toString.call(o).slice(8, -1); if (n === "Object" && o.constructor) n = o.constructor.name; if (n === "Map" || n === "Set") return Array.from(n); if (n === "Arguments" || /^(?:Ui|I)nt(?:8|16|32)(?:Clamped)?Array$/.test(n)) return _arrayLikeToArray(o, minLen); }
 
-function _arrayWithoutHoles(arr) { if (Array.isArray(arr)) { for (var i = 0, arr2 = new Array(arr.length); i < arr.length; i++) { arr2[i] = arr[i]; } return arr2; } }
+function _iterableToArray(iter) { if (typeof Symbol !== "undefined" && Symbol.iterator in Object(iter)) return Array.from(iter); }
 
+function _arrayWithoutHoles(arr) { if (Array.isArray(arr)) return _arrayLikeToArray(arr); }
+
+function _arrayLikeToArray(arr, len) { if (len == null || len > arr.length) len = arr.length; for (var i = 0, arr2 = new Array(len); i < len; i++) { arr2[i] = arr[i]; } return arr2; }
+
 var ruleName = (0, _utils.namespace)("at-mixin-named-arguments");
@@ -23,0 +27,0 @@ exports.ruleName = ruleName;

dist/rules/at-rule-conditional-no-parentheses/index.js

@@ -13,10 +13,14 @@ "use strict";
 
-function _toConsumableArray(arr) { return _arrayWithoutHoles(arr) || _iterableToArray(arr) || _nonIterableSpread(); }
+function _toConsumableArray(arr) { return _arrayWithoutHoles(arr) || _iterableToArray(arr) || _unsupportedIterableToArray(arr) || _nonIterableSpread(); }
 
-function _nonIterableSpread() { throw new TypeError("Invalid attempt to spread non-iterable instance"); }
+function _nonIterableSpread() { throw new TypeError("Invalid attempt to spread non-iterable instance.\nIn order to be iterable, non-array objects must have a [Symbol.iterator]() method."); }
 
-function _iterableToArray(iter) { if (Symbol.iterator in Object(iter) || Object.prototype.toString.call(iter) === "[object Arguments]") return Array.from(iter); }
+function _unsupportedIterableToArray(o, minLen) { if (!o) return; if (typeof o === "string") return _arrayLikeToArray(o, minLen); var n = Object.prototype.toString.call(o).slice(8, -1); if (n === "Object" && o.constructor) n = o.constructor.name; if (n === "Map" || n === "Set") return Array.from(n); if (n === "Arguments" || /^(?:Ui|I)nt(?:8|16|32)(?:Clamped)?Array$/.test(n)) return _arrayLikeToArray(o, minLen); }
 
-function _arrayWithoutHoles(arr) { if (Array.isArray(arr)) { for (var i = 0, arr2 = new Array(arr.length); i < arr.length; i++) { arr2[i] = arr[i]; } return arr2; } }
+function _iterableToArray(iter) { if (typeof Symbol !== "undefined" && Symbol.iterator in Object(iter)) return Array.from(iter); }
 
+function _arrayWithoutHoles(arr) { if (Array.isArray(arr)) return _arrayLikeToArray(arr); }
+
+function _arrayLikeToArray(arr, len) { if (len == null || len > arr.length) len = arr.length; for (var i = 0, arr2 = new Array(len); i < len; i++) { arr2[i] = arr[i]; } return arr2; }
+
 var ruleName = (0, _utils.namespace)("at-rule-conditional-no-parentheses");
@@ -23,0 +27,0 @@ exports.ruleName = ruleName;

dist/rules/dollar-variable-first-in-block/index.js

@@ -13,2 +13,8 @@ "use strict";
 
+function _createForOfIteratorHelper(o) { if (typeof Symbol === "undefined" || o[Symbol.iterator] == null) { if (Array.isArray(o) || (o = _unsupportedIterableToArray(o))) { var i = 0; var F = function F() {}; return { s: F, n: function n() { if (i >= o.length) return { done: true }; return { done: false, value: o[i++] }; }, e: function e(_e) { throw _e; }, f: F }; } throw new TypeError("Invalid attempt to iterate non-iterable instance.\nIn order to be iterable, non-array objects must have a [Symbol.iterator]() method."); } var it, normalCompletion = true, didErr = false, err; return { s: function s() { it = o[Symbol.iterator](); }, n: function n() { var step = it.next(); normalCompletion = step.done; return step; }, e: function e(_e2) { didErr = true; err = _e2; }, f: function f() { try { if (!normalCompletion && it["return"] != null) it["return"](); } finally { if (didErr) throw err; } } }; }
+
+function _unsupportedIterableToArray(o, minLen) { if (!o) return; if (typeof o === "string") return _arrayLikeToArray(o, minLen); var n = Object.prototype.toString.call(o).slice(8, -1); if (n === "Object" && o.constructor) n = o.constructor.name; if (n === "Map" || n === "Set") return Array.from(n); if (n === "Arguments" || /^(?:Ui|I)nt(?:8|16|32)(?:Clamped)?Array$/.test(n)) return _arrayLikeToArray(o, minLen); }
+
+function _arrayLikeToArray(arr, len) { if (len == null || len > arr.length) len = arr.length; for (var i = 0, arr2 = new Array(len); i < len; i++) { arr2[i] = arr[i]; } return arr2; }
+
 var ruleName = (0, _utils.namespace)("dollar-variable-first-in-block");
@@ -77,8 +83,8 @@ exports.ruleName = ruleName;
       var allowImports = (0, _utils.optionsHaveIgnored)(options, "imports");
-      var _iteratorNormalCompletion = true;
-      var _didIteratorError = false;
-      var _iteratorError = undefined;
 
+      var _iterator = _createForOfIteratorHelper(decl.parent.nodes),
+          _step;
+
       try {
-        for (var _iterator = decl.parent.nodes[Symbol.iterator](), _step; !(_iteratorNormalCompletion = (_step = _iterator.next()).done); _iteratorNormalCompletion = true) {
+        for (_iterator.s(); !(_step = _iterator.n()).done;) {
           var sibling = _step.value;
@@ -93,14 +99,5 @@
       } catch (err) {
-        _didIteratorError = true;
-        _iteratorError = err;
+        _iterator.e(err);
       } finally {
-        try {
-          if (!_iteratorNormalCompletion && _iterator["return"] != null) {
-            _iterator["return"]();
-          }
-        } finally {
-          if (_didIteratorError) {
-            throw _iteratorError;
-          }
-        }
+        _iterator.f();
       }
@@ -107,0 +104,0 @@

dist/rules/function-color-relative/index.js

@@ -26,2 +26,6 @@ "use strict";
 
+function isColorFunction(node) {
+  return node.type === "function" && function_names.includes(node.value);
+}
+
 function rule(primary) {
@@ -44,9 +48,16 @@ return function (root, result) {
 
-        if (function_names.includes(node.value)) {
-          _stylelint.utils.report({
-            message: messages.rejected,
-            node: decl,
-            index: (0, _utils.declarationValueIndex)(decl) + node.sourceIndex,
-            result: result,
-            ruleName: ruleName
+        var isFilter = decl.prop === "filter";
+        var isSassColorFunction = !isFilter && isColorFunction(node);
+        var isDSFilterColorFunction = isFilter && node.value === "drop-shadow" && node.nodes.some(isColorFunction);
+
+        if (isSassColorFunction || isDSFilterColorFunction) {
+          var nodes = isDSFilterColorFunction ? node.nodes.filter(isColorFunction) : [node];
+          nodes.forEach(function (node) {
+            _stylelint.utils.report({
+              message: messages.rejected,
+              node: decl,
+              index: (0, _utils.declarationValueIndex)(decl) + node.sourceIndex,
+              result: result,
+              ruleName: ruleName
+            });
           });
@@ -53,0 +64,0 @@ }

dist/rules/no-duplicate-dollar-variables/index.js

@@ -15,2 +15,8 @@ "use strict";
 
+function _createForOfIteratorHelper(o) { if (typeof Symbol === "undefined" || o[Symbol.iterator] == null) { if (Array.isArray(o) || (o = _unsupportedIterableToArray(o))) { var i = 0; var F = function F() {}; return { s: F, n: function n() { if (i >= o.length) return { done: true }; return { done: false, value: o[i++] }; }, e: function e(_e) { throw _e; }, f: F }; } throw new TypeError("Invalid attempt to iterate non-iterable instance.\nIn order to be iterable, non-array objects must have a [Symbol.iterator]() method."); } var it, normalCompletion = true, didErr = false, err; return { s: function s() { it = o[Symbol.iterator](); }, n: function n() { var step = it.next(); normalCompletion = step.done; return step; }, e: function e(_e2) { didErr = true; err = _e2; }, f: function f() { try { if (!normalCompletion && it["return"] != null) it["return"](); } finally { if (didErr) throw err; } } }; }
+
+function _unsupportedIterableToArray(o, minLen) { if (!o) return; if (typeof o === "string") return _arrayLikeToArray(o, minLen); var n = Object.prototype.toString.call(o).slice(8, -1); if (n === "Object" && o.constructor) n = o.constructor.name; if (n === "Map" || n === "Set") return Array.from(n); if (n === "Arguments" || /^(?:Ui|I)nt(?:8|16|32)(?:Clamped)?Array$/.test(n)) return _arrayLikeToArray(o, minLen); }
+
+function _arrayLikeToArray(arr, len) { if (len == null || len > arr.length) len = arr.length; for (var i = 0, arr2 = new Array(len); i < len; i++) { arr2[i] = arr[i]; } return arr2; }
+
 var ruleName = (0, _utils.namespace)("no-duplicate-dollar-variables");
@@ -53,8 +59,8 @@ exports.ruleName = ruleName;
       var scope = vars;
-      var _iteratorNormalCompletion = true;
-      var _didIteratorError = false;
-      var _iteratorError = undefined;
 
+      var _iterator = _createForOfIteratorHelper(ancestors),
+          _step;
+
       try {
-        for (var _iterator = ancestors[Symbol.iterator](), _step; !(_iteratorNormalCompletion = (_step = _iterator.next()).done); _iteratorNormalCompletion = true) {
+        for (_iterator.s(); !(_step = _iterator.n()).done;) {
           var node = _step.value;
@@ -69,14 +75,5 @@
       } catch (err) {
-        _didIteratorError = true;
-        _iteratorError = err;
+        _iterator.e(err);
       } finally {
-        try {
-          if (!_iteratorNormalCompletion && _iterator["return"] != null) {
-            _iterator["return"]();
-          }
-        } finally {
-          if (_didIteratorError) {
-            throw _iteratorError;
-          }
-        }
+        _iterator.f();
       }
@@ -94,8 +91,8 @@
       var scope = vars;
-      var _iteratorNormalCompletion2 = true;
-      var _didIteratorError2 = false;
-      var _iteratorError2 = undefined;
 
+      var _iterator2 = _createForOfIteratorHelper(ancestors),
+          _step2;
+
       try {
-        for (var _iterator2 = ancestors[Symbol.iterator](), _step2; !(_iteratorNormalCompletion2 = (_step2 = _iterator2.next()).done); _iteratorNormalCompletion2 = true) {
+        for (_iterator2.s(); !(_step2 = _iterator2.n()).done;) {
           var node = _step2.value;
@@ -106,14 +103,5 @@ scope = scope[node];
       } catch (err) {
-        _didIteratorError2 = true;
-        _iteratorError2 = err;
+        _iterator2.e(err);
       } finally {
-        try {
-          if (!_iteratorNormalCompletion2 && _iterator2["return"] != null) {
-            _iterator2["return"]();
-          }
-        } finally {
-          if (_didIteratorError2) {
-            throw _iteratorError2;
-          }
-        }
+        _iterator2.f();
       }
@@ -120,0 +108,0 @@

package.json

 {
   "name": "stylelint-scss",
   "description": "A collection of SCSS specific rules for stylelint",
-  "version": "3.17.0",
+  "version": "3.17.1",
   "author": "Krister Kari",
@@ -19,24 +19,24 @@ "babel": {
     "postcss-selector-parser": "^6.0.2",
-    "postcss-value-parser": "^4.0.2"
+    "postcss-value-parser": "^4.0.3"
   },
   "devDependencies": {
-    "@babel/cli": "^7.8.3",
-    "@babel/core": "^7.8.3",
-    "@babel/preset-env": "^7.8.3",
+    "@babel/cli": "^7.8.4",
+    "@babel/core": "^7.9.0",
+    "@babel/preset-env": "^7.9.5",
     "babel-plugin-istanbul": "^6.0.0",
-    "babel-preset-jest": "^25.1.0",
+    "babel-preset-jest": "^25.3.0",
     "eslint": "^6.8.0",
     "eslint-config-stylelint": "^11.1.0",
-    "eslint-plugin-lodash": "^6.0.0",
+    "eslint-plugin-lodash": "^7.1.0",
     "github-contributors-list": "^1.2.4",
-    "husky": "^4.2.0",
-    "jest": "^25.1.0",
-    "jest-cli": "^25.1.0",
-    "lint-staged": "^10.0.2",
+    "husky": "^4.2.5",
+    "jest": "^25.3.0",
+    "jest-cli": "^25.3.0",
+    "lint-staged": "^10.1.3",
     "npmpub": "^5.0.0",
-    "postcss": "^7.0.26",
+    "postcss": "^7.0.27",
     "postcss-scss": "^2.0.0",
     "prettier": "^1.19.1",
-    "rimraf": "^3.0.0",
-    "stylelint": "^13.0.0"
+    "rimraf": "^3.0.2",
+    "stylelint": "^13.3.2"
   },
@@ -43,0 +43,0 @@ "peerDependencies": {

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

356415

increased by1.03%

Worsened metrics

Lines of code

6286

decreased by-0.05%

Dependency changes

• + Addedcaniuse-lite@1.0.30001677(transitive)

• - Removedcaniuse-lite@1.0.30001678(transitive)

• Updatedpostcss-value-parser@^4.0.3