@primer/stylelint-config 12.3.1 → 12.3.3-rc.7590f39

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # Changelog
 
+## 12.3.3
+
+### Patch Changes
+
+- [#217](https://github.com/primer/stylelint-config/pull/217) [`5bb2834`](https://github.com/primer/stylelint-config/commit/5bb28342a6194dfdd4fbf5197682367ea54792b7) Thanks [@jsoref](https://github.com/jsoref)! - Spelling fixes
+
+## 12.3.2
+
+### Patch Changes
+
+- [#215](https://github.com/primer/stylelint-config/pull/215) [`66b16ae`](https://github.com/primer/stylelint-config/commit/66b16ae2edd81f8c8949f83c96d7011e5d395cc0) Thanks [@jonrohan](https://github.com/jonrohan)! - Making linter pick up separate function groups
+
 ## 12.3.1
 
 ### Patch Changes
@@ -24,7 +36,7 @@
 
 ### Patch Changes
 
-- [#181](https://github.com/primer/stylelint-config/pull/181) [`23e438a`](https://github.com/primer/stylelint-config/commit/23e438a7a9062550baa696cafbb186dc78b723f5) Thanks [@jonrohan](https://github.com/jonrohan)! - Truning off scss/dollar-variable-default
+- [#181](https://github.com/primer/stylelint-config/pull/181) [`23e438a`](https://github.com/primer/stylelint-config/commit/23e438a7a9062550baa696cafbb186dc78b723f5) Thanks [@jonrohan](https://github.com/jonrohan)! - Turning off scss/dollar-variable-default
 
 ## 12.1.1
 
@@ -86,7 +98,7 @@
 
 ### Major Changes
 
-- [`23a1f15`](https://github.com/primer/stylelint-config/commit/23a1f1599673f2a4f9f28c39da61f42871c05697) [#85](https://github.com/primer/stylelint-config/pull/85) Thanks [@koddsson](https://github.com/koddsson)! - Replace deprecated "blacklist" rules for "dissallow list" rules.
+- [`23a1f15`](https://github.com/primer/stylelint-config/commit/23a1f1599673f2a4f9f28c39da61f42871c05697) [#85](https://github.com/primer/stylelint-config/pull/85) Thanks [@koddsson](https://github.com/koddsson)! - Replace deprecated "blacklist" rules for "disallow list" rules.
 
   See https://stylelint.io/user-guide/rules/at-rule-blacklist and http://stylelint.io/user-guide/rules/declaration-property-value-disallowed-list/
 
@@ -104,7 +116,7 @@
 
 ### Patch Changes
 
-- [`d18cfbf`](https://github.com/primer/stylelint-config/commit/d18cfbfefc25be6ae38f73132552d2f3c62c4d02) [#79](https://github.com/primer/stylelint-config/pull/79) Thanks [@BinaryMuse](https://github.com/BinaryMuse)! - Add additinal verbose logging to `no-undefined-vars`
+- [`d18cfbf`](https://github.com/primer/stylelint-config/commit/d18cfbfefc25be6ae38f73132552d2f3c62c4d02) [#79](https://github.com/primer/stylelint-config/pull/79) Thanks [@BinaryMuse](https://github.com/BinaryMuse)! - Add additional verbose logging to `no-undefined-vars`
 
 * [`d18cfbf`](https://github.com/primer/stylelint-config/commit/d18cfbfefc25be6ae38f73132552d2f3c62c4d02) [#79](https://github.com/primer/stylelint-config/pull/79) Thanks [@BinaryMuse](https://github.com/BinaryMuse)! - Fix handling of edge-cases in `no-undefined-vars`
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@primer/stylelint-config",
-  "version": "12.3.1",
+  "version": "12.3.3-rc.7590f39",
   "description": "Sharable stylelint config used by GitHub's CSS",
   "homepage": "http://primer.style/css/tools/linting",
   "author": "GitHub, Inc.",
@@ -20,7 +20,7 @@
     "plugins/"
   ],
   "scripts": {
-    "test": "jest",
+    "test": "jest --coverage false",
     "lint": "eslint .",
     "release": "changeset publish"
   },

package/plugins/no-deprecated-colors.js

@@ -55,7 +55,7 @@ module.exports = stylelint.createPlugin(ruleName, (enabled, options = {}, contex
     }, {})
 
   const lintResult = (root, result) => {
-    // Walk all declarartions
+    // Walk all declarations
     root.walk(node => {
       if (seen.has(node)) {
         return

package/plugins/no-unused-vars.js

@@ -50,7 +50,7 @@ function getCachedVariables(options, log) {
   const key = JSON.stringify(options)
   return cache.tap(key, () => {
     const {files, variablePattern} = options
-    const decs = new TapMap()
+    const decls = new TapMap()
     const refs = new TapMap()
 
     log(`Looking for variables in ${files} ...`)
@@ -60,15 +60,15 @@ function getCachedVariables(options, log) {
         const after = css.substr(match.index + match[0].length)
         const name = match[0]
         if (after.startsWith(COLON)) {
-          decs.tap(name, set).add(file)
+          decls.tap(name, set).add(file)
         } else {
           refs.tap(name, set).add(file)
         }
       }
     }
-    log(`Found ${decs.size} declarations, ${pluralize(refs.size, 'reference')}.`)
+    log(`Found ${decls.size} declarations, ${pluralize(refs.size, 'reference')}.`)
 
-    for (const [name, filesList] of decs.entries()) {
+    for (const [name, filesList] of decls.entries()) {
       const fileRefs = refs.get(name)
       if (fileRefs) {
         log(`variable "${name}" declared in ${pluralize(filesList.size, 'file')}, ref'd in ${fileRefs.size}`)
@@ -77,7 +77,7 @@ function getCachedVariables(options, log) {
       }
     }
 
-    return {decs, refs}
+    return {decls, refs}
   })
 }
 

package/plugins/spacing.js

@@ -2,6 +2,7 @@ const stylelint = require('stylelint')
 const declarationValueIndex = require('stylelint/lib/utils/declarationValueIndex')
 const valueParser = require('postcss-value-parser')
 
+// TODO: Pull this in from primer/primitives
 const spacerValues = {
   '$spacer-1': '4px',
   '$spacer-2': '8px',
@@ -34,6 +35,17 @@ const messages = stylelint.utils.ruleMessages(ruleName, {
   }
 })
 
+const walkGroups = (root, validate) => {
+  for (const node of root.nodes) {
+    if (node.type === 'function') {
+      walkGroups(node, validate)
+    } else {
+      validate(node)
+    }
+  }
+  return root
+}
+
 // eslint-disable-next-line no-unused-vars
 module.exports = stylelint.createPlugin(ruleName, (enabled, options = {}, context) => {
   if (!enabled) {
@@ -47,63 +59,55 @@ module.exports = stylelint.createPlugin(ruleName, (enabled, options = {}, contex
       }
 
       const problems = []
-      let containsMath = false
-      let conatinsVariable = false
-      const parsedValue = valueParser(decl.value).walk(declValue => {
+
+      const parsedValue = walkGroups(valueParser(decl.value), node => {
+        // Remove leading negative sign, if any.
+        const cleanValue = node.value.replace(/^-/g, '')
+
         // Only check word types. https://github.com/TrySound/postcss-value-parser#word
-        if (!['word', 'function'].includes(declValue.type)) {
-          return false
+        if (node.type !== 'word') {
+          return
         }
 
-        // Ignore values that are not numbers.
-        if (['0', 'auto', 'inherit', 'initial'].includes(declValue.value)) {
-          return noop
+        // Exact values to ignore.
+        if (['*', '+', '-', '/', '0', 'auto', 'inherit', 'initial'].includes(node.value)) {
+          return
+        }
+
+        const valueUnit = valueParser.unit(cleanValue)
+
+        if (valueUnit && (valueUnit.unit === '' || !/^[0-9]+$/.test(valueUnit.number))) {
+          return
         }
-        // Remove leading negative sign, if any.
-        const cleanDeclValue = declValue.value.replace(/^-/g, '')
 
         // If the a variable is found in the value, skip it.
         if (
           Object.keys(spacerValues).some(variable =>
-            new RegExp(`${variable.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')}\\b`).test(cleanDeclValue)
+            new RegExp(`${variable.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')}\\b`).test(cleanValue)
           )
         ) {
-          conatinsVariable = true
-          return noop
+          return
         }
 
-        // For now we're going to ignore math.
-        if (['*', '+', '-', '/'].includes(declValue.value)) {
-          containsMath = true
-          return noop
-        }
+        const replacement = Object.keys(spacerValues).find(spacer => spacerValues[spacer] === cleanValue) || null
+        const valueMatch = replacement ? spacerValues[replacement] : node.value
 
-        let valueMatch = null
-        if ((valueMatch = Object.keys(spacerValues).find(spacer => spacerValues[spacer] === cleanDeclValue))) {
-          if (context.fix) {
-            declValue.value = declValue.value.replace(spacerValues[valueMatch], valueMatch)
-          } else {
-            problems.push({
-              index: declarationValueIndex(decl) + declValue.sourceIndex,
-              message: messages.rejected(spacerValues[valueMatch], valueMatch)
-            })
-          }
-        } else if (declValue.value !== '' && declValue.type !== 'function' && !containsMath) {
+        if (replacement && context.fix) {
+          node.value = node.value.replace(valueMatch, replacement)
+        } else {
           problems.push({
-            index: declarationValueIndex(decl) + declValue.sourceIndex,
-            message: messages.rejected(declValue.value, null)
+            index: declarationValueIndex(decl) + node.sourceIndex,
+            message: messages.rejected(valueMatch, replacement)
           })
         }
+
+        return
       })
 
       if (context.fix) {
         decl.value = parsedValue.toString()
       }
 
-      if (containsMath && conatinsVariable) {
-        return noop
-      }
-
       if (problems.length) {
         for (const err of problems) {
           stylelint.utils.report({