chore(eslint): Drop comma-dangle rule

kellnerd · kellnerd · commit 2fc11a22d275 · 2024-07-24T19:03:20.000+02:00
The rule to disallow trailing commas often forces you to pointlessly
touch neighboring lines when you add or remove items.

If we want to enforce consistent dangling commas, we should rather force
them to always be present. But since our whole codebase follows this
rule, it would be better to have a grace period to gradually add commas.
diff --git a/.eslintrc.js b/.eslintrc.js
@@ -204,7 +204,6 @@ const stylisticIssuesRules = {
 			properties: 'always'
 		}
 	],
-	'comma-dangle': ERROR,
 	'comma-spacing': ERROR,
 	'comma-style': ERROR,
 	'computed-property-spacing': ERROR,

Original file line number	Diff line number	Diff line change
`@@ -204,7 +204,6 @@ const stylisticIssuesRules = {`
`204`	`204`	`properties: 'always'`
`205`	`205`	`}`
`206`	`206`	`],`
`207`		`- 'comma-dangle': ERROR,`
`208`	`207`	`'comma-spacing': ERROR,`
`209`	`208`	`'comma-style': ERROR,`
`210`	`209`	`'computed-property-spacing': ERROR,`