fix: switch to @stylistic/exp-list-style to resolve array edge-cases

Signed-off-by: Ferdinand Thiessen <[email protected]>

Author: susnux

lib/configs/codeStyle.ts

@@ -93,25 +93,15 @@ export function codeStyle(options: ConfigOptions): Linter.Config[] {
 					{ avoidQuotes: true },
 				],
 
-				// Enforce consistent new lines after [ and before ]
-				'@stylistic/array-bracket-newline': [
-					'error',
-					'consistent',
-				],
-				// Enforce new lines between array elements (better git diff) but allow to have single line arrays
-				'@stylistic/array-element-newline': ['error', 'consistent'],
-				// Same for objects as for arrays
-				'@stylistic/object-curly-newline': [
-					'error',
-					{
-						consistent: true,
-						multiline: true,
-					},
-				],
-				'@stylistic/object-property-newline': [
-					'error',
-					{ allowAllPropertiesOnSameLine: true },
-				],
+				// Enforce consistent new lines after brackets
+				'@stylistic/array-bracket-newline': 'off',
+				'@stylistic/array-bracket-spacing': 'off',
+				'@stylistic/array-element-newline': 'off',
+				'@stylistic/jsx-function-call-newline': 'off',
+				'@stylistic/object-curly-newline': 'off',
+				'@stylistic/object-curly-spacing': 'off',
+				'@stylistic/object-property-newline': 'off',
+				'@stylistic/exp-list-style': 'error',
 
 				// No space between function name and parenthesis. Enforce fn() instead of fn ()
 				'@stylistic/function-call-spacing': [

lib/configs/vue.ts

@@ -143,7 +143,8 @@ export function vue(options: ConfigOptions): Linter.Config[] {
 				'vue/prefer-separate-static-class': 'error',
 				// For consistent layout of components
 				'vue/define-macros-order': [
-					'error', {
+					'error',
+					{
 						order: [
 							'defineOptions',
 							'defineModel',

tests/fixtures/codestyle/input/array.js

@@ -14,3 +14,22 @@ const arr3 = [
 	'second',
 	'third'
 ]
+
+// inconsistent brackets
+const brackets = ['Monday',
+	'Tuesday',
+	'Wednesday',
+	'Thursday',
+	'Friday',
+	'Saturday',
+	'Sunday',
+]
+
+// consistent brackets but not a list style
+const brackets2 = ['Monday',
+	'Tuesday',
+	'Wednesday',
+	'Thursday',
+	'Friday',
+	'Saturday',
+	'Sunday']

tests/fixtures/codestyle/output/array.js

@@ -17,3 +17,9 @@ const arr3 = [
 	'second',
 	'third',
 ]
+
+// inconsistent brackets
+const brackets = ['Monday',	'Tuesday',	'Wednesday',	'Thursday',	'Friday',	'Saturday',	'Sunday']
+
+// consistent brackets but not a list style
+const brackets2 = ['Monday',	'Tuesday',	'Wednesday',	'Thursday',	'Friday',	'Saturday',	'Sunday']

tests/fixtures/codestyle/output/function.ts

@@ -48,7 +48,10 @@ export function doSomething(
  * @param num
  * @param enable
  */
-export function doSomethingDifferent(num: number, enable: boolean) {
+export function doSomethingDifferent(
+	num: number,
+	enable: boolean,
+) {
 	// ...
 }