Merge pull request #350 from kristerkari/bugfix/map-keys-quotes-numer…

…ic-keys

map-keys-quotes: don't require numbers to be quoted

CHANGELOG.md

@@ -1,3 +1,7 @@
+# HEAD
+
+- Fixed: `map-keys-quotes` warning for unquoted numeric keys.
+
 # 3.9.2
 
 - Fixed: `selector-no-union-class-name` throwing an error when using nested `@`-rules.

src/rules/map-keys-quotes/__tests__/index.js

@@ -1,4 +1,4 @@
-import rule, { ruleName, messages } from "..";
+import rule, { messages, ruleName } from "..";
 
 testRule(rule, {
   ruleName,
@@ -11,6 +11,40 @@ testRule(rule, {
         $test: ("foo": 14px, "bar": 25px);
       `,
       description: "accepts strings without quotes"
+    },
+    {
+      code: `
+        $test: ('foo': 14px, 'bar': 25px);
+      `,
+      description: "accepts strings with quotes"
+    },
+    {
+      code: `
+      $colors: (
+        250: #eef9ff,
+        300: #bbe7ff,
+        350: #b7d3e6,
+        500: #1388db,
+        750: #0f6bac,
+        900: #2e4662
+      );
+      `,
+      description: "accepts numbers"
+    },
+    {
+      code: `
+      $colors: (
+        'primary': (
+          250: #eef9ff,
+          300: #bbe7ff,
+          350: #b7d3e6,
+          500: #1388db,
+          750: #0f6bac,
+          900: #2e4662,
+        )
+      );
+      `,
+      description: "accepts numbers (nested)"
     }
   ],
 

src/rules/map-keys-quotes/index.js

@@ -1,6 +1,6 @@
+import valueParser from "postcss-value-parser";
 import { utils } from "stylelint";
 import { namespace } from "../../utils";
-import valueParser from "postcss-value-parser";
 
 export const ruleName = namespace("map-keys-quotes");
 
@@ -34,7 +34,7 @@ function rule(primary) {
           const mapKeys = returnMapKeys(node.nodes);
 
           mapKeys.forEach(map_key => {
-            if (map_key.type === "word") {
+            if (map_key.type === "word" && isNaN(map_key.value)) {
               utils.report({
                 message: messages.expected,
                 node: decl,