Skip to content

Commit 246e77c

Browse files
mydeamydea
committed
chore: Ensure prettier is run on all files
This simplifies our prettier setup to also run on e2e-tests, and aligns the `fix:prettier` and `lint:prettier` targets (which were previously not aligned).
1 parent 499b79a commit 246e77c

File tree

181 files changed

+1981
-1472
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

181 files changed

+1981
-1472
lines changed

‎.claude/settings.local.json‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,4 +11,4 @@
1111
],
1212
"deny": []
1313
}
14-
}
14+
}

‎.prettierignore‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
11
packages/browser/test/loader.js
22
packages/replay-worker/examples/worker.min.js
33
dev-packages/browser-integration-tests/fixtures
4+
**/test.ts-snapshots/**

‎.prettierrc.json‎

Lines changed: 0 additions & 7 deletions
This file was deleted.

‎dev-packages/cloudflare-integration-tests/tsconfig.json‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,6 @@
77
// Although this seems wrong to include `DOM` here, it's necessary to make
88
// global fetch available in tests in lower Node versions.
99
"lib": ["ES2020"],
10-
"esModuleInterop": true,
10+
"esModuleInterop": true
1111
}
1212
}

‎dev-packages/e2e-tests/test-applications/angular-17/angular.json‎

Lines changed: 6 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -16,17 +16,10 @@
1616
"outputPath": "dist/angular-17",
1717
"index": "src/index.html",
1818
"browser": "src/main.ts",
19-
"polyfills": [
20-
"zone.js"
21-
],
19+
"polyfills": ["zone.js"],
2220
"tsConfig": "tsconfig.app.json",
23-
"assets": [
24-
"src/favicon.ico",
25-
"src/assets"
26-
],
27-
"styles": [
28-
"src/styles.css"
29-
],
21+
"assets": ["src/favicon.ico", "src/assets"],
22+
"styles": ["src/styles.css"],
3023
"scripts": []
3124
},
3225
"configurations": {
@@ -74,18 +67,10 @@
7467
"test": {
7568
"builder": "@angular-devkit/build-angular:karma",
7669
"options": {
77-
"polyfills": [
78-
"zone.js",
79-
"zone.js/testing"
80-
],
70+
"polyfills": ["zone.js", "zone.js/testing"],
8171
"tsConfig": "tsconfig.spec.json",
82-
"assets": [
83-
"src/favicon.ico",
84-
"src/assets"
85-
],
86-
"styles": [
87-
"src/styles.css"
88-
],
72+
"assets": ["src/favicon.ico", "src/assets"],
73+
"styles": ["src/styles.css"],
8974
"scripts": []
9075
}
9176
}

‎dev-packages/e2e-tests/test-applications/angular-17/src/app/component-tracking/component-tracking.components.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,8 @@ import { SampleComponent } from '../sample-component/sample-component.components
77
standalone: true,
88
imports: [TraceModule, SampleComponent],
99
template: `
10-
<app-sample-component [trace]="'sample-component'"></app-sample-component>
11-
<app-sample-component trace></app-sample-component>
10+
<app-sample-component [trace]="'sample-component'"></app-sample-component>
11+
<app-sample-component trace></app-sample-component>
1212
`,
1313
})
1414
@TraceClass({ name: 'ComponentTrackingComponent' })

‎dev-packages/e2e-tests/test-applications/angular-17/src/app/home/home.component.ts‎

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -6,18 +6,18 @@ import { RouterLink } from '@angular/router';
66
standalone: true,
77
imports: [RouterLink],
88
template: `
9-
<main>
10-
<h1>Welcome to Sentry's Angular 17 E2E test app</h1>
11-
<ul>
12-
<li> <a id="navLink" [routerLink]="['/users', '123']">Visit User 123</a> </li>
13-
<li> <a id="redirectLink" [routerLink]="['/redirect1']">Redirect</a> </li>
14-
<li> <a id="cancelLink" [routerLink]="['/cancel']">Cancel</a> </li>
15-
<li> <a id="nonExistentLink" [routerLink]="['/non-existent']">Error</a> </li>
16-
<li> <a id="componentTracking" [routerLink]="['/component-tracking']">Component Tracking</a> </li>
17-
</ul>
18-
<button id="errorBtn" (click)="throwError()">Throw error</button>
19-
</main>
20-
`,
9+
<main>
10+
<h1>Welcome to Sentry's Angular 17 E2E test app</h1>
11+
<ul>
12+
<li><a id="navLink" [routerLink]="['/users', '123']">Visit User 123</a></li>
13+
<li><a id="redirectLink" [routerLink]="['/redirect1']">Redirect</a></li>
14+
<li><a id="cancelLink" [routerLink]="['/cancel']">Cancel</a></li>
15+
<li><a id="nonExistentLink" [routerLink]="['/non-existent']">Error</a></li>
16+
<li><a id="componentTracking" [routerLink]="['/component-tracking']">Component Tracking</a></li>
17+
</ul>
18+
<button id="errorBtn" (click)="throwError()">Throw error</button>
19+
</main>
20+
`,
2121
})
2222
export class HomeComponent {
2323
throwError() {

‎dev-packages/e2e-tests/test-applications/angular-17/tsconfig.json‎

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -19,10 +19,7 @@
1919
"target": "ES2022",
2020
"module": "ES2022",
2121
"useDefineForClassFields": false,
22-
"lib": [
23-
"ES2022",
24-
"dom"
25-
]
22+
"lib": ["ES2022", "dom"]
2623
},
2724
"angularCompilerOptions": {
2825
"enableI18nLegacyMessageIdFormat": false,

‎dev-packages/e2e-tests/test-applications/angular-18/angular.json‎

Lines changed: 6 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -16,17 +16,10 @@
1616
"outputPath": "dist/angular-18",
1717
"index": "src/index.html",
1818
"browser": "src/main.ts",
19-
"polyfills": [
20-
"zone.js"
21-
],
19+
"polyfills": ["zone.js"],
2220
"tsConfig": "tsconfig.app.json",
23-
"assets": [
24-
"src/favicon.ico",
25-
"src/assets"
26-
],
27-
"styles": [
28-
"src/styles.css"
29-
],
21+
"assets": ["src/favicon.ico", "src/assets"],
22+
"styles": ["src/styles.css"],
3023
"scripts": []
3124
},
3225
"configurations": {
@@ -74,18 +67,10 @@
7467
"test": {
7568
"builder": "@angular-devkit/build-angular:karma",
7669
"options": {
77-
"polyfills": [
78-
"zone.js",
79-
"zone.js/testing"
80-
],
70+
"polyfills": ["zone.js", "zone.js/testing"],
8171
"tsConfig": "tsconfig.spec.json",
82-
"assets": [
83-
"src/favicon.ico",
84-
"src/assets"
85-
],
86-
"styles": [
87-
"src/styles.css"
88-
],
72+
"assets": ["src/favicon.ico", "src/assets"],
73+
"styles": ["src/styles.css"],
8974
"scripts": []
9075
}
9176
}

‎dev-packages/e2e-tests/test-applications/angular-18/src/app/home/home.component.ts‎

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -6,18 +6,18 @@ import { RouterLink } from '@angular/router';
66
standalone: true,
77
imports: [RouterLink],
88
template: `
9-
<main>
10-
<h1>Welcome to Sentry's Angular 18 E2E test app</h1>
11-
<ul>
12-
<li> <a id="navLink" [routerLink]="['/users', '123']">Visit User 123</a> </li>
13-
<li> <a id="redirectLink" [routerLink]="['/redirect1']">Redirect</a> </li>
14-
<li> <a id="cancelLink" [routerLink]="['/cancel']">Cancel</a> </li>
15-
<li> <a id="nonExistentLink" [routerLink]="['/non-existent']">Error</a> </li>
16-
<li> <a id="componentTracking" [routerLink]="['/component-tracking']">Component Tracking</a> </li>
17-
</ul>
18-
<button id="errorBtn" (click)="throwError()">Throw error</button>
19-
</main>
20-
`,
9+
<main>
10+
<h1>Welcome to Sentry's Angular 18 E2E test app</h1>
11+
<ul>
12+
<li><a id="navLink" [routerLink]="['/users', '123']">Visit User 123</a></li>
13+
<li><a id="redirectLink" [routerLink]="['/redirect1']">Redirect</a></li>
14+
<li><a id="cancelLink" [routerLink]="['/cancel']">Cancel</a></li>
15+
<li><a id="nonExistentLink" [routerLink]="['/non-existent']">Error</a></li>
16+
<li><a id="componentTracking" [routerLink]="['/component-tracking']">Component Tracking</a></li>
17+
</ul>
18+
<button id="errorBtn" (click)="throwError()">Throw error</button>
19+
</main>
20+
`,
2121
})
2222
export class HomeComponent {
2323
throwError() {

0 commit comments

Comments
 (0)