Disable three more rules to simplify Stylelint 14 upgrades

Author: thibaudcolas

CHANGELOG.md

@@ -6,6 +6,16 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
 
 ## Unreleased
 
+## [2.0.3](https://github.com/torchbox/stylelint-config-torchbox/compare/v2.0.2...v2.0.3) (2023-08-17)
+
+### Bug fixes
+
+Disables the following rules introduced in Stylelint 14 & related package updates, to make it simpler for projects to upgrade to Stylelint 14:
+
+- [`declaration-block-no-redundant-longhand-properties`](https://stylelint.io/user-guide/rules/declaration-block-no-redundant-longhand-properties/)
+- [`selector-id-pattern`](https://stylelint.io/user-guide/rules/selector-id-pattern/)
+- [`shorthand-property-no-redundant-values`](https://stylelint.io/user-guide/rules/shorthand-property-no-redundant-values/)
+
 ## [2.0.2](https://github.com/torchbox/stylelint-config-torchbox/compare/v2.0.1...v2.0.2) (2023-08-17)
 
 ### Bug fixes

README.md

@@ -151,7 +151,6 @@ To get the most out of this config, it is assumed that projects have the followi
 - [`comment-whitespace-inside`](https://stylelint.io/user-guide/rules/comment-whitespace-inside/): `always`
 - [`custom-property-empty-line-before`](https://stylelint.io/user-guide/rules/custom-property-empty-line-before/): `always, except: after-custom-property, first-nested, ignore: after-comment, inside-single-line-block`
 - [`declaration-block-single-line-max-declarations`](https://stylelint.io/user-guide/rules/declaration-block-single-line-max-declarations/): `1`
-- [`declaration-block-no-redundant-longhand-properties`](https://stylelint.io/user-guide/rules/declaration-block-no-redundant-longhand-properties/)
 - [`font-family-name-quotes`](https://stylelint.io/user-guide/rules/font-family-name-quotes/): `always-where-recommended`
 - [`function-name-case`](https://stylelint.io/user-guide/rules/function-name-case/): `lower`
 - [`function-url-quotes`](https://stylelint.io/user-guide/rules/function-url-quotes/): `always`
@@ -163,10 +162,8 @@ To get the most out of this config, it is assumed that projects have the followi
 - [`no-irregular-whitespace`](https://stylelint.io/user-guide/rules/no-irregular-whitespace/)
 - [`number-max-precision`](https://stylelint.io/user-guide/rules/number-max-precision/): `4`
 - [`rule-empty-line-before`](https://stylelint.io/user-guide/rules/rule-empty-line-before/): `always-multi-line, except: first-nested, ignore: after-comment`
-- [`selector-id-pattern`](https://stylelint.io/user-guide/rules/selector-id-pattern/): `^(a-za-z0-9*)(-a-z0-9+)*$, message: Expected id selector to be kebab-case`
 - [`selector-pseudo-element-colon-notation`](https://stylelint.io/user-guide/rules/selector-pseudo-element-colon-notation/): `double`
 - [`selector-type-case`](https://stylelint.io/user-guide/rules/selector-type-case/): `lower`
-- [`shorthand-property-no-redundant-values`](https://stylelint.io/user-guide/rules/shorthand-property-no-redundant-values/)
 - [`value-keyword-case`](https://stylelint.io/user-guide/rules/value-keyword-case/): `lower`
 - [`block-no-empty`](https://stylelint.io/user-guide/rules/block-no-empty/)
 - [`color-no-invalid-hex`](https://stylelint.io/user-guide/rules/color-no-invalid-hex/)

config.js

@@ -32,10 +32,11 @@ module.exports = {
             { resolveNestedSelectors: true },
         ],
         'keyframes-name-pattern': null,
-        'alpha-value-notation': null,
-        'color-function-notation': null,
         'custom-media-pattern': null,
         'custom-property-pattern': null,
+        'selector-id-pattern': null,
+        'alpha-value-notation': null,
+        'color-function-notation': null,
         'max-nesting-depth': 4,
         'selector-max-specificity': '0,4,0',
         'value-no-vendor-prefix': true,
@@ -46,6 +47,8 @@ module.exports = {
         'no-descending-specificity': null,
         'at-rule-no-vendor-prefix': true,
         'at-rule-no-unknown': null,
+        'declaration-block-no-redundant-longhand-properties': null,
+        'shorthand-property-no-redundant-values': null,
         // Replaces previously-used `a11y/no-text-align-justify`.
         'declaration-property-value-disallowed-list': {
             'text-align': ['justify'],

src/__snapshots__/semver.test.js.snap

@@ -84,9 +84,7 @@ exports[`semver - should those tests break, consider releasing a new major versi
       ],
     },
   ],
-  "declaration-block-no-redundant-longhand-properties": [
-    true,
-  ],
+  "declaration-block-no-redundant-longhand-properties": null,
   "declaration-block-no-shorthand-property-overrides": [
     true,
   ],
@@ -323,12 +321,7 @@ exports[`semver - should those tests break, consider releasing a new major versi
   "selector-combinator-space-after": null,
   "selector-combinator-space-before": null,
   "selector-descendant-combinator-no-non-space": null,
-  "selector-id-pattern": [
-    "^([a-z][a-z0-9]*)(-[a-z0-9]+)*$",
-    {
-      "message": "Expected id selector to be kebab-case",
-    },
-  ],
+  "selector-id-pattern": null,
   "selector-list-comma-newline-after": null,
   "selector-list-comma-newline-before": null,
   "selector-list-comma-space-after": null,
@@ -376,9 +369,7 @@ exports[`semver - should those tests break, consider releasing a new major versi
       ],
     },
   ],
-  "shorthand-property-no-redundant-values": [
-    true,
-  ],
+  "shorthand-property-no-redundant-values": null,
   "string-no-newline": [
     true,
   ],

src/rules.md

@@ -36,7 +36,6 @@
 - [`comment-whitespace-inside`](https://stylelint.io/user-guide/rules/comment-whitespace-inside/): `always`
 - [`custom-property-empty-line-before`](https://stylelint.io/user-guide/rules/custom-property-empty-line-before/): `always, except: after-custom-property, first-nested, ignore: after-comment, inside-single-line-block`
 - [`declaration-block-single-line-max-declarations`](https://stylelint.io/user-guide/rules/declaration-block-single-line-max-declarations/): `1`
-- [`declaration-block-no-redundant-longhand-properties`](https://stylelint.io/user-guide/rules/declaration-block-no-redundant-longhand-properties/)
 - [`font-family-name-quotes`](https://stylelint.io/user-guide/rules/font-family-name-quotes/): `always-where-recommended`
 - [`function-name-case`](https://stylelint.io/user-guide/rules/function-name-case/): `lower`
 - [`function-url-quotes`](https://stylelint.io/user-guide/rules/function-url-quotes/): `always`
@@ -48,10 +47,8 @@
 - [`no-irregular-whitespace`](https://stylelint.io/user-guide/rules/no-irregular-whitespace/)
 - [`number-max-precision`](https://stylelint.io/user-guide/rules/number-max-precision/): `4`
 - [`rule-empty-line-before`](https://stylelint.io/user-guide/rules/rule-empty-line-before/): `always-multi-line, except: first-nested, ignore: after-comment`
-- [`selector-id-pattern`](https://stylelint.io/user-guide/rules/selector-id-pattern/): `^(a-za-z0-9*)(-a-z0-9+)*$, message: Expected id selector to be kebab-case`
 - [`selector-pseudo-element-colon-notation`](https://stylelint.io/user-guide/rules/selector-pseudo-element-colon-notation/): `double`
 - [`selector-type-case`](https://stylelint.io/user-guide/rules/selector-type-case/): `lower`
-- [`shorthand-property-no-redundant-values`](https://stylelint.io/user-guide/rules/shorthand-property-no-redundant-values/)
 - [`value-keyword-case`](https://stylelint.io/user-guide/rules/value-keyword-case/): `lower`
 - [`block-no-empty`](https://stylelint.io/user-guide/rules/block-no-empty/)
 - [`color-no-invalid-hex`](https://stylelint.io/user-guide/rules/color-no-invalid-hex/)
@@ -83,13 +80,16 @@
 > Rules that are turned off on purpose.
 
 - [`keyframes-name-pattern`](https://stylelint.io/user-guide/rules/keyframes-name-pattern/)
-- [`alpha-value-notation`](https://stylelint.io/user-guide/rules/alpha-value-notation/)
-- [`color-function-notation`](https://stylelint.io/user-guide/rules/color-function-notation/)
 - [`custom-media-pattern`](https://stylelint.io/user-guide/rules/custom-media-pattern/)
 - [`custom-property-pattern`](https://stylelint.io/user-guide/rules/custom-property-pattern/)
+- [`selector-id-pattern`](https://stylelint.io/user-guide/rules/selector-id-pattern/)
+- [`alpha-value-notation`](https://stylelint.io/user-guide/rules/alpha-value-notation/)
+- [`color-function-notation`](https://stylelint.io/user-guide/rules/color-function-notation/)
 - [`declaration-empty-line-before`](https://stylelint.io/user-guide/rules/declaration-empty-line-before/)
 - [`no-descending-specificity`](https://stylelint.io/user-guide/rules/no-descending-specificity/)
 - [`at-rule-no-unknown`](https://stylelint.io/user-guide/rules/at-rule-no-unknown/)
+- [`declaration-block-no-redundant-longhand-properties`](https://stylelint.io/user-guide/rules/declaration-block-no-redundant-longhand-properties/)
+- [`shorthand-property-no-redundant-values`](https://stylelint.io/user-guide/rules/shorthand-property-no-redundant-values/)
 - [`at-rule-empty-line-before`](https://stylelint.io/user-guide/rules/at-rule-empty-line-before/)
 - [`at-rule-name-case`](https://stylelint.io/user-guide/rules/at-rule-name-case/)
 - [`at-rule-name-space-after`](https://stylelint.io/user-guide/rules/at-rule-name-space-after/)