♻️ [Eslint] Adjust rules for latest antfu

beefchimi · Dec 6, 2024 · e01827b · e01827b
1 parent 8a8fee6
commit e01827b
Show file tree

Hide file tree

Showing 3 changed files with 32 additions and 2 deletions.
diff --git a/app/website/eslint.config.mjs b/app/website/eslint.config.mjs
@@ -37,13 +37,20 @@ export default antfu(
   {
     // Without a `files` scope, these rules will apply to everything.
     rules: {
-      'antfu/if-newline': 'off',
+      // General
       'no-console': 'warn',
+
+      // Antfu
+      'antfu/if-newline': 'off',
+      'perfectionist/sort-exports': 'off',
+      'perfectionist/sort-named-exports': 'off',
+      'perfectionist/sort-imports': 'off',
       'style/arrow-parens': ['error', 'always'],
       'style/object-curly-spacing': ['error', 'never'],
       'test/prefer-lowercase-title': 'off',
       'ts/explicit-function-return-type': 'off',
       'ts/strict-boolean-expressions': 'off',
+
       // Would like this if I could differentiate between ternary and if conditions.
       // 'style/operator-linebreak': ['error', 'after'],
 

diff --git a/eslint.config.mjs b/eslint.config.mjs
@@ -25,13 +25,20 @@ export default antfu(
   {
     // Without a `files` scope, these rules will apply to everything.
     rules: {
-      'antfu/if-newline': 'off',
+      // General
       'no-console': 'warn',
+
+      // Antfu
+      'antfu/if-newline': 'off',
+      'perfectionist/sort-exports': 'off',
+      'perfectionist/sort-named-exports': 'off',
+      'perfectionist/sort-imports': 'off',
       'style/arrow-parens': ['error', 'always'],
       'style/object-curly-spacing': ['error', 'never'],
       'test/prefer-lowercase-title': 'off',
       'ts/explicit-function-return-type': 'off',
       'ts/strict-boolean-expressions': 'off',
+
       // Would like this if I could differentiate between ternary and if conditions.
       // 'style/operator-linebreak': ['error', 'after'],
 

diff --git a/pkg/earwurm/src/tests/Earwurm.test.ts b/pkg/earwurm/src/tests/Earwurm.test.ts
@@ -560,9 +560,25 @@ describe('Earwurm component', () => {
         }
       });
 
+      const readySounds = await Promise.all(sounds);
+
+      for (const sound of readySounds) {
+        sound.play();
+      }
+
+      /*
+      Alternate #1:
       for await (const sound of sounds) {
         sound.play();
       }
+      */
+
+      /*
+      Alternate #2:
+      for (const sound of sounds) {
+        (await sound).play();
+      }
+      */
 
       expect(mockManager.playing).toBe(true);
       expect(mockManager.keys).toHaveLength(3);