Ignore interpolated properties in property-no-unknown (#977)

stylelint-scss · Mar 9, 2024 · 4513387 · 4513387
1 parent 735aece
commit 4513387
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 0 deletions.
diff --git a/src/rules/property-no-unknown/__tests__/index.js b/src/rules/property-no-unknown/__tests__/index.js
@@ -5,6 +5,7 @@ const { messages, ruleName } = require("..");
 testRule({
   ruleName,
   config: [true],
+  customSyntax: "postcss-scss",
 
   accept: [
     {
@@ -37,6 +38,13 @@ testRule({
     {
       code: ".foo { --custom-property-set: { colr: blue; } }",
       description: "ignore custom property sets"
+    },
+    {
+      code: `
+      $custom-property-set: '--custom-property-set';
+      .b { #{$custom-property-set}: blue; }
+      `,
+      description: "ignore custom property sets"
     }
   ],
 

diff --git a/src/rules/property-no-unknown/index.js b/src/rules/property-no-unknown/index.js
@@ -7,6 +7,7 @@ const isType = require("../../utils/isType");
 const optionsMatches = require("../../utils/optionsMatches");
 const namespace = require("../../utils/namespace");
 const ruleUrl = require("../../utils/ruleUrl");
+const hasInterpolation = require("../../utils/hasInterpolation");
 const properties = require("known-css-properties").all;
 
 const { utils } = require("stylelint");
@@ -67,6 +68,7 @@ function rule(primary, secondaryOptions) {
         !isStandardSyntaxDeclaration(decl) ||
         isCustomPropertySet(prop) ||
         prop.startsWith("--") ||
+        hasInterpolation(prop) ||
         (!shouldCheckPrefixed && vendorPrefix(prop)) ||
         optionsMatches(secondaryOptions, "ignoreProperties", prop)
       ) {