Skip to content

Commit 7a699a6

Browse files
chore: remove formatting-related lint rules internally (#19473)
* chore: remove formatting-related lint rules internally * Also disable MD010 and MD030 in preparation for 19355 * Also disable MD013 in preparation for 19355 * Updated tools/update-rule-type-headers.js to print tabs
1 parent c99db89 commit 7a699a6

11 files changed

+1930
-1947
lines changed

.markdownlint.yml

+1-7
Original file line numberDiff line numberDiff line change
@@ -2,16 +2,10 @@ default: true
22

33
# Exclusions for deliberate/widespread violations
44
MD002: false # First header should be a h1 header
5-
MD004: # Unordered list style
6-
style: asterisk
7-
MD007: # Unordered list indentation
8-
indent: 4
5+
MD010: false # Hard tabs
96
MD013: false # Line length
10-
MD019: false # Multiple spaces after hash on atx style header
11-
MD021: false # Multiple spaces inside hashes on closed atx style header
127
MD024: false # Multiple headers with the same content
138
MD026: false # Trailing punctuation in header
14-
MD029: false # Ordered list item prefix
159
MD030: false # Spaces after list markers
1610
MD033: false # Allow inline HTML
1711
MD041: false # First line in file should be a top level header

docs/.stylelintrc.json

+14-4
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
{
2-
"extends": ["stylelint-config-standard-scss"],
2+
"extends": ["stylelint-config-standard-scss", "stylelint-config-prettier"],
33
"rules": {
44
"alpha-value-notation": "number",
55
"at-rule-empty-line-before": null,
@@ -11,8 +11,6 @@
1111
}],
1212
"declaration-block-no-redundant-longhand-properties": null,
1313
"hue-degree-notation": "number",
14-
"indentation": 4,
15-
"max-line-length": null,
1614
"no-descending-specificity": null,
1715
"number-leading-zero": null,
1816
"number-no-trailing-zeros": null,
@@ -24,7 +22,19 @@
2422
"files": [
2523
"**/*.html"
2624
],
27-
"extends": ["stylelint-config-html/html", "stylelint-config-standard"]
25+
"extends": [
26+
"stylelint-config-html/html",
27+
"stylelint-config-standard",
28+
"stylelint-config-prettier"
29+
]
30+
},
31+
{
32+
"files": [
33+
"**/*.scss"
34+
],
35+
"rules": {
36+
"scss/operator-no-newline-after": null
37+
}
2838
}
2939
],
3040
"ignoreFiles": [

docs/package.json

+1
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,7 @@
5353
"stylelint-config-html": "^1.1.0",
5454
"stylelint-config-standard": "^29.0.0",
5555
"stylelint-config-standard-scss": "^5.0.0",
56+
"stylelint-config-prettier": "^9.0.5",
5657
"tap-spot": "^1.1.2"
5758
},
5859
"engines": {

eslint.config.js

+2-7
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,6 @@ const eslintPluginRulesRecommendedConfig = require("eslint-plugin-eslint-plugin/
1515
const eslintPluginTestsRecommendedConfig = require("eslint-plugin-eslint-plugin/configs/tests-recommended");
1616
const globals = require("globals");
1717
const eslintConfigESLintCJS = require("eslint-config-eslint/cjs");
18-
const eslintConfigESLintFormatting = require("eslint-config-eslint/formatting");
1918
const eslintPluginYml = require("eslint-plugin-yml");
2019
const json = require("@eslint/json").default;
2120
const expectType = require("eslint-plugin-expect-type");
@@ -80,11 +79,6 @@ module.exports = defineConfig([
8079
files: [ALL_JS_FILES],
8180
extends: [eslintConfigESLintCJS]
8281
},
83-
{
84-
name: "eslint/formatting",
85-
files: [ALL_JS_FILES],
86-
extends: [eslintConfigESLintFormatting]
87-
},
8882
globalIgnores([
8983
"build/**",
9084
"coverage/**",
@@ -159,7 +153,8 @@ module.exports = defineConfig([
159153
"errors"
160154
]
161155
],
162-
"eslint-plugin/test-case-shorthand-strings": "error"
156+
"eslint-plugin/test-case-shorthand-strings": "error",
157+
"no-useless-concat": "off"
163158
}
164159
},
165160
{

0 commit comments

Comments
 (0)