Merge pull request #3124 from modernweb-dev/chore/improve-prettier-setup

chore: improve prettier setup

Author: bashmish

.eslintignore

@@ -19,3 +19,4 @@ _site
 _site-dev
 dist-types
 /docs/_merged_data
+**/__snapshots__/**

docs/docs/building/rollup-plugin-import-meta-assets.md

@@ -98,9 +98,7 @@ In this example, we use it to optimize SVG images with [svgo](https://github.com
 import { importMetaAssets } from '@web/rollup-plugin-import-meta-assets';
 const svgo = new SVGO({
   // See https://github.com/svg/svgo#what-it-can-do
-  plugins: [
-    /* plugins here */
-  ],
+  plugins: [/* plugins here */],
 });
 
 export default {

docs/docs/dev-server/middleware.md

@@ -44,8 +44,8 @@ export default {
 };
 ```
 
-</details>  
-  
+</details>
+
 ## Rewriting request urls
 
 You can rewrite certain file requests using a middleware. This can be useful for example to serve your `index.html` from a different file location or to alias a module.

docs/docs/dev-server/plugins/esbuild.md

@@ -32,16 +32,7 @@ We expose the following options for esbuild. Most of them are a mirror of the es
 
 ```ts
 type Loader =
-  | 'js'
-  | 'jsx'
-  | 'ts'
-  | 'tsx'
-  | 'json'
-  | 'text'
-  | 'base64'
-  | 'file'
-  | 'dataurl'
-  | 'binary';
+  'js' | 'jsx' | 'ts' | 'tsx' | 'json' | 'text' | 'base64' | 'file' | 'dataurl' | 'binary';
 
 interface EsbuildPluginArgs {
   target?: string | string[];

docs/docs/dev-server/writing-plugins/overview.md

@@ -38,13 +38,9 @@ import { Server } from 'net';
 import { DevServerCoreConfig, Logger, WebSocketsManager } from '@web/dev-server-core';
 
 export type ServeResult =
-  | void
-  | string
-  | { body: string; type?: string; headers?: Record<string, string> };
+  void | string | { body: string; type?: string; headers?: Record<string, string> };
 export type TransformResult =
-  | void
-  | string
-  | { body?: string; headers?: Record<string, string>; transformCache?: boolean };
+  void | string | { body?: string; headers?: Record<string, string>; transformCache?: boolean };
 export type ResolveResult = void | string | { id?: string };
 export type ResolveMimeTypeResult = void | string | { type?: string };
 

integration/test-runner/index.ts

@@ -1,11 +1,11 @@
 import type { BrowserLauncher, TestRunnerCoreConfig } from '@web/test-runner-core';
 import { runBasicTest } from './tests/basic/runBasicTest.ts';
 import { runConfigGroupsTest } from './tests/config-groups/runConfigGroupsTest.ts';
-import { runParallelTest } from './tests/parallel/runParallelTest.ts';
-import { runTestFailureTest } from './tests/test-failure/runTestFailureTest.ts';
-import { runLocationChangeTest } from './tests/location-change/runLocationChangeTest.ts';
 import { runFocusTest } from './tests/focus/runFocusTest.ts';
+import { runLocationChangeTest } from './tests/location-change/runLocationChangeTest.ts';
 import { runManyTests } from './tests/many/runManyTests.ts';
+import { runParallelTest } from './tests/parallel/runParallelTest.ts';
+import { runTestFailureTest } from './tests/test-failure/runTestFailureTest.ts';
 
 export interface Tests {
   basic: boolean;

integration/test-runner/tests/basic/browser-tests/timers.test.js

@@ -2,7 +2,7 @@ import { expect } from '../../../../../node_modules/@esm-bundle/chai/esm/chai.js
 
 describe('timers test', () => {
   it('can call setTimeout', async () => {
-    const promise = new Promise((resolve) => {
+    const promise = new Promise(resolve => {
       window.setTimeout(() => {
         resolve();
       }, 0);
@@ -20,7 +20,7 @@ describe('timers test', () => {
 
     window.clearTimeout(timer);
 
-    await new Promise((resolve) => {
+    await new Promise(resolve => {
       window.setTimeout(() => resolve(), 2);
     });
 
@@ -34,7 +34,7 @@ describe('timers test', () => {
       callCount++;
     }, 1);
 
-    await new Promise((resolve) => {
+    await new Promise(resolve => {
       window.setTimeout(() => resolve(), 2);
     });
 
@@ -44,15 +44,15 @@ describe('timers test', () => {
 
     window.clearInterval(interval);
 
-    await new Promise((resolve) => {
+    await new Promise(resolve => {
       window.setTimeout(() => resolve(), 2);
     });
 
     expect(callCount).to.equal(oldCallCount);
   });
 
   it('can call requestAnimationFrame', async () => {
-    const promise = new Promise((resolve) => {
+    const promise = new Promise(resolve => {
       window.requestAnimationFrame(() => {
         resolve();
       });
@@ -71,7 +71,7 @@ describe('timers test', () => {
 
     window.cancelAnimationFrame(timer);
 
-    await new Promise((resolve) => {
+    await new Promise(resolve => {
       window.setTimeout(() => resolve(), 2);
     });
 

integration/test-runner/tests/basic/runBasicTest.ts

@@ -1,8 +1,8 @@
-import assert from 'node:assert/strict';
-import { describe, before, it } from 'node:test';
+import { legacyPlugin } from '@web/dev-server-legacy';
 import type { BrowserLauncher, TestRunnerCoreConfig, TestSession } from '@web/test-runner-core';
 import { runTests } from '@web/test-runner-core/test-helpers';
-import { legacyPlugin } from '@web/dev-server-legacy';
+import assert from 'node:assert/strict';
+import { before, describe, it } from 'node:test';
 import { resolve } from 'path';
 
 export function runBasicTest(

integration/test-runner/tests/config-groups/runConfigGroupsTest.ts

@@ -1,12 +1,12 @@
-import assert from 'node:assert/strict';
-import { describe, it } from 'node:test';
+import { legacyPlugin } from '@web/dev-server-legacy';
 import type {
   BrowserLauncher,
   TestRunnerCoreConfig,
   TestRunnerGroupConfig,
 } from '@web/test-runner-core';
 import { runTests } from '@web/test-runner-core/test-helpers';
-import { legacyPlugin } from '@web/dev-server-legacy';
+import assert from 'node:assert/strict';
+import { describe, it } from 'node:test';
 import { resolve } from 'path';
 
 export function runConfigGroupsTest(

integration/test-runner/tests/focus/runFocusTest.ts

@@ -1,8 +1,8 @@
-import assert from 'node:assert/strict';
-import { describe, before, it } from 'node:test';
+import { legacyPlugin } from '@web/dev-server-legacy';
 import type { BrowserLauncher, TestRunnerCoreConfig, TestSession } from '@web/test-runner-core';
 import { runTests } from '@web/test-runner-core/test-helpers';
-import { legacyPlugin } from '@web/dev-server-legacy';
+import assert from 'node:assert/strict';
+import { before, describe, it } from 'node:test';
 import { resolve } from 'path';
 
 export function runFocusTest(