@primer/stylelint-config 12.7.2 → 12.7.3-rc.99e67f7

package/plugins/lib/variable-rules.js

@@ -10,7 +10,7 @@ module.exports = {
   createVariableRule,
   CSS_DIRECTIONS,
   CSS_CORNERS,
-  CSS_IMPORTANT
+  CSS_IMPORTANT,
 }
 
 function createVariableRule(ruleName, rules, url) {
@@ -79,7 +79,7 @@ function createVariableRule(ruleName, rules, url) {
                       return `${m}. See ${url}.`
                     }
                     return `${m}.`
-                  }
+                  },
                 })
                 .rejected(error)
 
@@ -87,7 +87,7 @@ function createVariableRule(ruleName, rules, url) {
                 message,
                 node: decl,
                 result,
-                ruleName
+                ruleName,
               })
             }
           }

package/plugins/no-deprecated-colors.js

@@ -9,7 +9,7 @@ const messages = stylelint.utils.ruleMessages(ruleName, {
     }
 
     return `Variable ${varName} is deprecated for property ${property}. Please use the replacement ${replacement}.`
-  }
+  },
 })
 
 // Match CSS variable references (e.g var(--color-text-primary))
@@ -84,7 +84,7 @@ module.exports = stylelint.createPlugin(ruleName, (enabled, options = {}, contex
             message: messages.rejected(variableName, replacement, node.prop),
             node,
             ruleName,
-            result
+            result,
           })
         }
       }
@@ -92,7 +92,7 @@ module.exports = stylelint.createPlugin(ruleName, (enabled, options = {}, contex
   }
 
   log(
-    `${Object.keys(replacedVars).length} deprecated variables replaced with ${Object.keys(newVars).length} variables.`
+    `${Object.keys(replacedVars).length} deprecated variables replaced with ${Object.keys(newVars).length} variables.`,
   )
   return lintResult
 })

package/plugins/no-experimental-vars.js

@@ -5,7 +5,7 @@ const ruleName = 'primer/no-experimental-vars'
 const messages = stylelint.utils.ruleMessages(ruleName, {
   rejected: value => {
     return `Do not use experimental variable \`var(--${value})\`. Experimental variables are undergoing testing, see https://github.com/github/primer/issues/889 for more details.`
-  }
+  },
 })
 
 // eslint-disable-next-line no-unused-vars
@@ -31,7 +31,7 @@ module.exports = stylelint.createPlugin(ruleName, (enabled, options = {}, contex
             message: messages.rejected(expVar),
             node: decl,
             result,
-            ruleName
+            ruleName,
           })
         }
       }

package/plugins/no-override.js

@@ -52,7 +52,7 @@ module.exports = stylelint.createPlugin(ruleName, (enabled, options = {}) => {
       const ruleSelector = collapseWhitespace(rule.selector)
       const context = selector === rule.selector ? '' : ` in "${ruleSelector}"`
       return `"${collapseWhitespace(selector)}" should not be overridden${context}${definedIn}.`
-    }
+    },
   })
 
   return (root, result) => {
@@ -62,7 +62,7 @@ module.exports = stylelint.createPlugin(ruleName, (enabled, options = {}) => {
         : '(not an array)'
       result.warn(`The "bundles" option must be an array of valid bundles; got: ${invalidBundles}`, {
         stylelintType: 'invalidOption',
-        stylelintReference: 'https://github.com/primer/stylelint-config#options'
+        stylelintReference: 'https://github.com/primer/stylelint-config#options',
       })
     }
 
@@ -71,7 +71,7 @@ module.exports = stylelint.createPlugin(ruleName, (enabled, options = {}) => {
         message: messages.rejected(subject),
         node: subject.rule,
         result,
-        ruleName
+        ruleName,
       })
 
     root.walkRules(rule => {
@@ -82,7 +82,7 @@ module.exports = stylelint.createPlugin(ruleName, (enabled, options = {}) => {
             return report({
               rule,
               bundle: immutableSelectors.get(selector),
-              selector
+              selector,
             })
           }
         } else {
@@ -95,7 +95,7 @@ module.exports = stylelint.createPlugin(ruleName, (enabled, options = {}) => {
             return report({
               rule,
               bundle: immutableClassSelectors.get(classSelector),
-              selector: classSelector
+              selector: classSelector,
             })
           }
         }

package/plugins/no-scale-colors.js

@@ -4,7 +4,7 @@ const matchAll = require('string.prototype.matchall')
 const ruleName = 'primer/no-scale-colors'
 const messages = stylelint.utils.ruleMessages(ruleName, {
   rejected: varName =>
-    `${varName} is a non-functional scale color and cannot be used without being wrapped in the color-variables mixin`
+    `${varName} is a non-functional scale color and cannot be used without being wrapped in the color-variables mixin`,
 })
 
 // Match CSS variable references (e.g var(--color-text-primary))
@@ -39,7 +39,7 @@ module.exports = stylelint.createPlugin(ruleName, (enabled, options = {}) => {
               message: messages.rejected(variableName),
               node: decl,
               result,
-              ruleName
+              ruleName,
             })
           }
         }

package/plugins/no-undefined-vars.js

@@ -6,7 +6,7 @@ const TapMap = require('tap-map')
 
 const ruleName = 'primer/no-undefined-vars'
 const messages = stylelint.utils.ruleMessages(ruleName, {
-  rejected: varName => `${varName} is not defined`
+  rejected: varName => `${varName} is not defined`,
 })
 
 // Match CSS variable definitions (e.g. --color-text-primary:)
@@ -40,7 +40,7 @@ module.exports = stylelint.createPlugin(ruleName, (enabled, options = {}) => {
           message: messages.rejected(variableName),
           node,
           result,
-          ruleName
+          ruleName,
         })
       }
     }
@@ -80,7 +80,7 @@ module.exports = stylelint.createPlugin(ruleName, (enabled, options = {}) => {
           checkVariable(
             variableName,
             decl,
-            new Set([...globalDefinedVariables, ...fileDefinedVariables, ...scopeDefinedVaribles])
+            new Set([...globalDefinedVariables, ...fileDefinedVariables, ...scopeDefinedVaribles]),
           )
         }
       })

package/plugins/no-unused-vars.js

@@ -11,7 +11,7 @@ const COLON = ':'
 const SCSS_VARIABLE_PATTERN = /(\$[-\w]+)/g
 
 const messages = stylelint.utils.ruleMessages(ruleName, {
-  rejected: name => `The variable "${name}" is not referenced.`
+  rejected: name => `The variable "${name}" is not referenced.`,
 })
 
 const cache = new TapMap()
@@ -35,7 +35,7 @@ module.exports = stylelint.createPlugin(ruleName, (enabled, options = {}) => {
             message: messages.rejected(name),
             node: decl,
             result,
-            ruleName
+            ruleName,
           })
         } else {
           const path = stripCwd(decl.source.input.file)

package/plugins/responsive-widths.js

@@ -7,7 +7,7 @@ const ruleName = 'primer/responsive-widths'
 const messages = stylelint.utils.ruleMessages(ruleName, {
   rejected: value => {
     return `A value larger than the smallest viewport could break responsive pages. Use a width value smaller than ${value}. https://primer.style/css/support/breakpoints`
-  }
+  },
 })
 
 // 320px is the smallest viewport size that we support
@@ -60,7 +60,7 @@ module.exports = stylelint.createPlugin(ruleName, (enabled, options = {}, contex
             if (parseInt(valueUnit.number) > 320) {
               problems.push({
                 index: declarationValueIndex(decl) + node.sourceIndex,
-                message: messages.rejected(node.value)
+                message: messages.rejected(node.value),
               })
             }
             break
@@ -68,7 +68,7 @@ module.exports = stylelint.createPlugin(ruleName, (enabled, options = {}, contex
             if (parseInt(valueUnit.number) > 100) {
               problems.push({
                 index: declarationValueIndex(decl) + node.sourceIndex,
-                message: messages.rejected(node.value)
+                message: messages.rejected(node.value),
               })
             }
             break
@@ -82,7 +82,7 @@ module.exports = stylelint.createPlugin(ruleName, (enabled, options = {}, contex
             message: err.message,
             node: decl,
             result,
-            ruleName
+            ruleName,
           })
         }
       }

package/plugins/spacing.js

@@ -21,7 +21,7 @@ const spacerValues = {
   '$em-spacer-3': '0.25em',
   '$em-spacer-4': '0.375em',
   '$em-spacer-5': '0.5em',
-  '$em-spacer-6': '0.75em'
+  '$em-spacer-6': '0.75em',
 }
 
 const ruleName = 'primer/spacing'
@@ -32,7 +32,7 @@ const messages = stylelint.utils.ruleMessages(ruleName, {
     }
 
     return `Please replace ${value} with spacing variable '${replacement}'.`
-  }
+  },
 })
 
 const walkGroups = (root, validate) => {
@@ -83,7 +83,7 @@ module.exports = stylelint.createPlugin(ruleName, (enabled, options = {}, contex
         // If the a variable is found in the value, skip it.
         if (
           Object.keys(spacerValues).some(variable =>
-            new RegExp(`${variable.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')}\\b`).test(cleanValue)
+            new RegExp(`${variable.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')}\\b`).test(cleanValue),
           )
         ) {
           return
@@ -97,7 +97,7 @@ module.exports = stylelint.createPlugin(ruleName, (enabled, options = {}, contex
         } else {
           problems.push({
             index: declarationValueIndex(decl) + node.sourceIndex,
-            message: messages.rejected(valueMatch, replacement)
+            message: messages.rejected(valueMatch, replacement),
           })
         }
 
@@ -115,7 +115,7 @@ module.exports = stylelint.createPlugin(ruleName, (enabled, options = {}, contex
             message: err.message,
             node: decl,
             result,
-            ruleName
+            ruleName,
           })
         }
       }

package/plugins/typography.js

@@ -5,20 +5,20 @@ module.exports = createVariableRule(
   {
     'font-size': {
       expects: 'a font-size variable',
-      values: ['$body-font-size', '$h{000,00,0,1,2,3,4,5,6}-size', '$font-size-*', '1', '1em', 'inherit']
+      values: ['$body-font-size', '$h{000,00,0,1,2,3,4,5,6}-size', '$font-size-*', '1', '1em', 'inherit'],
     },
     'font-weight': {
       props: 'font-weight',
       values: ['$font-weight-*', 'inherit'],
       replacements: {
         bold: '$font-weight-bold',
-        normal: '$font-weight-normal'
-      }
+        normal: '$font-weight-normal',
+      },
     },
     'line-height': {
       props: 'line-height',
-      values: ['$body-line-height', '$lh-*', '0', '1', '1em', 'inherit']
-    }
+      values: ['$body-line-height', '$lh-*', '0', '1', '1em', 'inherit'],
+    },
   },
-  'https://primer.style/css/utilities/typography'
+  'https://primer.style/css/utilities/typography',
 )

package/plugins/utilities.js

@@ -6,7 +6,7 @@ const ruleName = 'primer/utilities'
 const messages = stylelint.utils.ruleMessages(ruleName, {
   rejected: (selector, utilityClass) => {
     return `Consider using the Primer utility '.${utilityClass}' instead of the selector '${selector}' in your html. https://primer.style/css/utilities`
-  }
+  },
 })
 
 // eslint-disable-next-line no-unused-vars
@@ -39,7 +39,7 @@ module.exports = stylelint.createPlugin(ruleName, (enabled, options = {}, contex
             message: messages.rejected(rule.selector, replacement.utilityClass),
             node: rule,
             result,
-            ruleName
+            ruleName,
           })
         }
       }

package/property-order.js

@@ -168,5 +168,5 @@ module.exports = [
   'animation-timing-function',
   'animation-delay',
   'animation-iteration-count',
-  'animation-direction'
+  'animation-direction',
 ]