fix: stylelint rule disables

primer · Jan 30, 2025 · 0a5a817 · 0a5a817
1 parent 0ae0893
commit 0a5a817
Show file tree

Hide file tree

Showing 11 changed files with 21 additions and 8 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -46,7 +46,7 @@
     "@csstools/postcss-sass": "^5.1.1",
     "@github/prettier-config": "^0.0.6",
     "@primer/primitives": "^10.3.4",
-    "@primer/stylelint-config": "^13.2.2",
+    "@primer/stylelint-config": "^13.2.3",
     "@primer/view-components": "^0.36.5",
     "autoprefixer": "^10.4.18",
     "chokidar-cli": "^3.0.0",

diff --git a/src/base/kbd.scss b/src/base/kbd.scss
@@ -4,7 +4,7 @@
 kbd {
   display: inline-block;
   padding: var(--base-size-4);
-  // stylelint-disable-next-line primer/typography
+  // stylelint-disable-next-line primer/typography, declaration-property-value-no-unknown
   font: 11px $mono-font;
   // stylelint-disable-next-line primer/typography
   line-height: 10px;

diff --git a/src/branch-name/branch-name.scss b/src/branch-name/branch-name.scss
@@ -6,7 +6,7 @@
   display: inline-block;
   // stylelint-disable-next-line primer/spacing
   padding: 2px 6px;
-  // stylelint-disable-next-line primer/typography
+  // stylelint-disable-next-line primer/typography, declaration-property-value-no-unknown
   font: 12px $mono-font;
   color: var(--fgColor-muted, var(--color-fg-muted));
   word-break: break-all;

diff --git a/src/layout/grid.scss b/src/layout/grid.scss
@@ -1,16 +1,28 @@
 // GRID
 
 // Columns
+
+// stylelint-disable-next-line  declaration-property-value-no-unknown
 .col-1 { width: (1 * 0.0833333333 * 100%); }
+// stylelint-disable-next-line  declaration-property-value-no-unknown
 .col-2 { width: (2 * 0.0833333333 * 100%); }
+// stylelint-disable-next-line  declaration-property-value-no-unknown
 .col-3 { width: (3 * 0.0833333333 * 100%); }
+// stylelint-disable-next-line  declaration-property-value-no-unknown
 .col-4 { width: (4 * 0.0833333333 * 100%); }
+// stylelint-disable-next-line  declaration-property-value-no-unknown
 .col-5 { width: (5 * 0.0833333333 * 100%); }
+// stylelint-disable-next-line  declaration-property-value-no-unknown
 .col-6 { width: (6 * 0.0833333333 * 100%); }
+// stylelint-disable-next-line  declaration-property-value-no-unknown
 .col-7 { width: (7 * 0.0833333333 * 100%); }
+// stylelint-disable-next-line  declaration-property-value-no-unknown
 .col-8 { width: (8 * 0.0833333333 * 100%); }
+// stylelint-disable-next-line  declaration-property-value-no-unknown
 .col-9 { width: (9 * 0.0833333333 * 100%); }
+// stylelint-disable-next-line  declaration-property-value-no-unknown
 .col-10 { width: (10 * 0.0833333333 * 100%); }
+// stylelint-disable-next-line  declaration-property-value-no-unknown
 .col-11 { width: (11 * 0.0833333333 * 100%); }
 .col-12 { width: 100%; }
 

diff --git a/src/markdown/code.scss b/src/markdown/code.scss
@@ -68,7 +68,6 @@
   pre code,
   pre tt {
     display: inline;
-    max-width: auto;
     padding: 0;
     margin: 0;
     overflow: visible;

diff --git a/src/markdown/footnotes.scss b/src/markdown/footnotes.scss
@@ -41,7 +41,7 @@
       left: calc(var(--base-size-24) * -1);
       pointer-events: none;
       content: '';
-      // stylelint-disable-next-line primer/borders, primer/colors
+      // stylelint-disable-next-line primer/borders, primer/colors, declaration-property-value-no-unknown
       border: 2px $border-style var(--borderColor-accent-emphasis, var(--color-accent-emphasis));
       // stylelint-disable-next-line primer/borders
       border-radius: $border-radius;

diff --git a/src/markdown/markdown-body.scss b/src/markdown/markdown-body.scss
@@ -87,7 +87,7 @@
     // stylelint-disable-next-line primer/spacing
     padding: 0 1em;
     color: var(--fgColor-muted, var(--color-fg-muted));
-    // stylelint-disable-next-line primer/borders, primer/colors
+    // stylelint-disable-next-line primer/borders, primer/colors, declaration-property-value-no-unknown
     border-left: 0.25em $border-style var(--borderColor-default, var(--color-border-default));
 
     > :first-child {

diff --git a/src/marketing/utilities/animations.scss b/src/marketing/utilities/animations.scss
@@ -1,6 +1,7 @@
 // Animation utilities for marketing
 
 .hover-grow-mktg {
+  // stylelint-disable-next-line declaration-property-value-no-unknown
   transition: transform 0.4s $ease-mktg;
 
   &:hover {

diff --git a/src/navigation/sidenav.scss b/src/navigation/sidenav.scss
@@ -25,7 +25,7 @@
 
   &:last-child {
     // makes sure there is a "bottom border" in case the list is not long enough
-    // stylelint-disable-next-line primer/box-shadow
+    // stylelint-disable-next-line primer/box-shadow, declaration-property-value-no-unknown
     box-shadow: 0 $border-width 0 var(--borderColor-default, var(--color-border-default));
   }
 

diff --git a/src/utilities/typography.scss b/src/utilities/typography.scss
@@ -287,6 +287,7 @@
 
 /* Force long "words" to wrap if they exceed the width of the container */
 .wb-break-word {
+    // stylelint-disable-next-line declaration-property-value-keyword-no-deprecated
   word-break: break-word !important;
   // this is for backwards compatibility with browsers that don't respect overflow-wrap
   word-wrap: break-word !important;