From 309ceb8418c4b8cb8d45786ad13531d26e1efb85 Mon Sep 17 00:00:00 2001 From: Seppe Dekeyser Date: Mon, 19 Aug 2024 22:30:45 +0200 Subject: [PATCH] Bump eslint in e2e/kit, update erroring old svelte syntax --- e2e/kit/package.json | 8 +- e2e/kit/src/lib/QueryExt.svelte | 6 +- e2e/kit/src/params/integer.js | 1 + e2e/kit/src/routes/lists/all/+page.svelte | 7 +- .../plugin/subscription/renders/+page.svelte | 4 +- .../src/routes/stores/metadata/+page.svelte | 5 +- .../routes/stores/mutation-update/+page.ts | 4 +- .../routes/stores/nested-list/+page.svelte | 6 +- .../+page.svelte | 9 +- .../src/routes/stores/network/+page.svelte | 8 +- .../query/forward-cursor/+page.svelte | 6 +- .../routes/stores/subunsub-list/+page.svelte | 6 +- e2e/react/playwright.config.ts | 1 + e2e/svelte/playwright.config.ts | 6 +- pnpm-lock.yaml | 128 ++++++++---------- 15 files changed, 103 insertions(+), 102 deletions(-) diff --git a/e2e/kit/package.json b/e2e/kit/package.json index 23681fc4c6..0541db9fbf 100644 --- a/e2e/kit/package.json +++ b/e2e/kit/package.json @@ -28,14 +28,14 @@ "@sveltejs/adapter-auto": "^3.2.1", "@sveltejs/kit": "^2.5.10", "@sveltejs/vite-plugin-svelte": "^3.1.1", - "@typescript-eslint/eslint-plugin": "^7.11.0", - "@typescript-eslint/parser": "^7.11.0", + "@typescript-eslint/eslint-plugin": "^8.2.0", + "@typescript-eslint/parser": "^8.2.0", "concurrently": "7.1.0", "cross-env": "^7.0.3", "e2e-api": "workspace:^", "eslint": "^8.57.0", "eslint-config-prettier": "^9.1.0", - "eslint-plugin-svelte": "^2.39.0", + "eslint-plugin-svelte": "^2.43.0", "houdini": "workspace:^", "houdini-plugin-svelte-global-stores": "workspace:^", "houdini-svelte": "workspace:^", @@ -43,7 +43,7 @@ "prettier-plugin-svelte": "^3.2.3", "svelte": "5.0.0-next.208", "svelte-check": "^3.8.0", - "svelte-eslint-parser": "^0.36.0", + "svelte-eslint-parser": "^0.41.0", "svelte-preprocess": "^5.1.4", "tslib": "^2.3.1", "typescript": "^5.0.0", diff --git a/e2e/kit/src/lib/QueryExt.svelte b/e2e/kit/src/lib/QueryExt.svelte index cb0c4bb223..13501c5234 100644 --- a/e2e/kit/src/lib/QueryExt.svelte +++ b/e2e/kit/src/lib/QueryExt.svelte @@ -1,8 +1,10 @@

diff --git a/e2e/kit/src/params/integer.js b/e2e/kit/src/params/integer.js index 1b6695388f..00928d5afe 100644 --- a/e2e/kit/src/params/integer.js +++ b/e2e/kit/src/params/integer.js @@ -1,5 +1,6 @@ import { MyEnum } from '$houdini'; // this is here to silence the warning that MyEnum isn't being used :facepalm: +// eslint-disable-next-line MyEnum; /** @type {import('@sveltejs/kit').ParamMatcher} */ diff --git a/e2e/kit/src/routes/lists/all/+page.svelte b/e2e/kit/src/routes/lists/all/+page.svelte index d2f2bd64b4..32a0cdd9ff 100644 --- a/e2e/kit/src/routes/lists/all/+page.svelte +++ b/e2e/kit/src/routes/lists/all/+page.svelte @@ -1,8 +1,8 @@

hello
diff --git a/e2e/kit/src/routes/stores/metadata/+page.svelte b/e2e/kit/src/routes/stores/metadata/+page.svelte index c97e4f0a18..5a74373c39 100644 --- a/e2e/kit/src/routes/stores/metadata/+page.svelte +++ b/e2e/kit/src/routes/stores/metadata/+page.svelte @@ -1,6 +1,6 @@

Metadata

diff --git a/e2e/kit/src/routes/stores/mutation-update/+page.ts b/e2e/kit/src/routes/stores/mutation-update/+page.ts index bcff404d98..c3384dca31 100644 --- a/e2e/kit/src/routes/stores/mutation-update/+page.ts +++ b/e2e/kit/src/routes/stores/mutation-update/+page.ts @@ -7,7 +7,7 @@ export async function load(event: LoadEvent) { }; } -graphql` +graphql(` query TestMutationUpdateUsersList { usersList(limit: 5, snapshot: "update-user-mutation") { id @@ -15,4 +15,4 @@ graphql` ...UserInfo } } -`; +`); diff --git a/e2e/kit/src/routes/stores/nested-list/+page.svelte b/e2e/kit/src/routes/stores/nested-list/+page.svelte index cfc7510f29..8d118e4a3e 100644 --- a/e2e/kit/src/routes/stores/nested-list/+page.svelte +++ b/e2e/kit/src/routes/stores/nested-list/+page.svelte @@ -1,5 +1,4 @@

network-one-store-multivariables

diff --git a/e2e/kit/src/routes/stores/network/+page.svelte b/e2e/kit/src/routes/stores/network/+page.svelte index 1566fa0439..44d6a90c22 100644 --- a/e2e/kit/src/routes/stores/network/+page.svelte +++ b/e2e/kit/src/routes/stores/network/+page.svelte @@ -1,9 +1,11 @@

NETWORK

diff --git a/e2e/kit/src/routes/stores/pagination/query/forward-cursor/+page.svelte b/e2e/kit/src/routes/stores/pagination/query/forward-cursor/+page.svelte index ec28ca3852..5cc54f981d 100644 --- a/e2e/kit/src/routes/stores/pagination/query/forward-cursor/+page.svelte +++ b/e2e/kit/src/routes/stores/pagination/query/forward-cursor/+page.svelte @@ -1,8 +1,10 @@

Sub Unsub - List

diff --git a/e2e/react/playwright.config.ts b/e2e/react/playwright.config.ts index 814f9f959f..2b965c01c0 100644 --- a/e2e/react/playwright.config.ts +++ b/e2e/react/playwright.config.ts @@ -6,6 +6,7 @@ export default defineConfig({ reporter: process.env.CI ? [['list'], ['html'], ['github']] : [['list']], use: { screenshot: 'only-on-failure' }, testIgnore: '**/$houdini/**', + testMatch: 'test.ts', webServer: { command: 'PORT=3008 npm run dev', diff --git a/e2e/svelte/playwright.config.ts b/e2e/svelte/playwright.config.ts index e3bb63ad9f..66027baf23 100644 --- a/e2e/svelte/playwright.config.ts +++ b/e2e/svelte/playwright.config.ts @@ -1,4 +1,4 @@ -import { defineConfig } from "@playwright/test"; +import { defineConfig } from '@playwright/test' // eslint-disable-next-line @typescript-eslint/no-explicit-any const reporter = [['list']] @@ -9,7 +9,7 @@ if (process.env.CI) { const config = defineConfig({ retries: process.env.CI ? 3 : 0, - testMatch: 'spec.ts', + testMatch: 'spec.ts', workers: 5, reporter, screenshot: 'only-on-failure', @@ -18,6 +18,6 @@ const config = defineConfig({ port: 3006, timeout: 120 * 1000, }, -}); +}) export default config diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml index e062f7569f..5e0f5a0334 100644 --- a/pnpm-lock.yaml +++ b/pnpm-lock.yaml @@ -134,11 +134,11 @@ importers: specifier: ^3.1.1 version: 3.1.1(svelte@5.0.0-next.208)(vite@5.3.3) '@typescript-eslint/eslint-plugin': - specifier: ^7.11.0 - version: 7.18.0(@typescript-eslint/parser@7.18.0)(eslint@8.57.0)(typescript@5.4.2) + specifier: ^8.2.0 + version: 8.2.0(@typescript-eslint/parser@8.2.0)(eslint@8.57.0)(typescript@5.4.2) '@typescript-eslint/parser': - specifier: ^7.11.0 - version: 7.18.0(eslint@8.57.0)(typescript@5.4.2) + specifier: ^8.2.0 + version: 8.2.0(eslint@8.57.0)(typescript@5.4.2) concurrently: specifier: 7.1.0 version: 7.1.0 @@ -155,7 +155,7 @@ importers: specifier: ^9.1.0 version: 9.1.0(eslint@8.57.0) eslint-plugin-svelte: - specifier: ^2.39.0 + specifier: ^2.43.0 version: 2.43.0(eslint@8.57.0)(svelte@5.0.0-next.208) houdini: specifier: workspace:^ @@ -179,8 +179,8 @@ importers: specifier: ^3.8.0 version: 3.8.0(@babel/core@7.20.7)(postcss@8.4.39)(svelte@5.0.0-next.208) svelte-eslint-parser: - specifier: ^0.36.0 - version: 0.36.0(svelte@5.0.0-next.208) + specifier: ^0.41.0 + version: 0.41.0(svelte@5.0.0-next.208) svelte-preprocess: specifier: ^5.1.4 version: 5.1.4(@babel/core@7.20.7)(postcss@8.4.39)(svelte@5.0.0-next.208)(typescript@5.4.2) @@ -3978,23 +3978,23 @@ packages: - supports-color dev: true - /@typescript-eslint/eslint-plugin@7.18.0(@typescript-eslint/parser@7.18.0)(eslint@8.57.0)(typescript@5.4.2): - resolution: {integrity: sha512-94EQTWZ40mzBc42ATNIBimBEDltSJ9RQHCC8vc/PDbxi4k8dVwUAv4o98dk50M1zB+JGFxp43FP7f8+FP8R6Sw==} - engines: {node: ^18.18.0 || >=20.0.0} + /@typescript-eslint/eslint-plugin@8.2.0(@typescript-eslint/parser@8.2.0)(eslint@8.57.0)(typescript@5.4.2): + resolution: {integrity: sha512-02tJIs655em7fvt9gps/+4k4OsKULYGtLBPJfOsmOq1+3cdClYiF0+d6mHu6qDnTcg88wJBkcPLpQhq7FyDz0A==} + engines: {node: ^18.18.0 || ^20.9.0 || >=21.1.0} peerDependencies: - '@typescript-eslint/parser': ^7.0.0 - eslint: ^8.56.0 + '@typescript-eslint/parser': ^8.0.0 || ^8.0.0-alpha.0 + eslint: ^8.57.0 || ^9.0.0 typescript: '*' peerDependenciesMeta: typescript: optional: true dependencies: '@eslint-community/regexpp': 4.10.0 - '@typescript-eslint/parser': 7.18.0(eslint@8.57.0)(typescript@5.4.2) - '@typescript-eslint/scope-manager': 7.18.0 - '@typescript-eslint/type-utils': 7.18.0(eslint@8.57.0)(typescript@5.4.2) - '@typescript-eslint/utils': 7.18.0(eslint@8.57.0)(typescript@5.4.2) - '@typescript-eslint/visitor-keys': 7.18.0 + '@typescript-eslint/parser': 8.2.0(eslint@8.57.0)(typescript@5.4.2) + '@typescript-eslint/scope-manager': 8.2.0 + '@typescript-eslint/type-utils': 8.2.0(eslint@8.57.0)(typescript@5.4.2) + '@typescript-eslint/utils': 8.2.0(eslint@8.57.0)(typescript@5.4.2) + '@typescript-eslint/visitor-keys': 8.2.0 eslint: 8.57.0 graphemer: 1.4.0 ignore: 5.3.1 @@ -4045,20 +4045,20 @@ packages: - supports-color dev: true - /@typescript-eslint/parser@7.18.0(eslint@8.57.0)(typescript@5.4.2): - resolution: {integrity: sha512-4Z+L8I2OqhZV8qA132M4wNL30ypZGYOQVBfMgxDH/K5UX0PNqTu1c6za9ST5r9+tavvHiTWmBnKzpCJ/GlVFtg==} - engines: {node: ^18.18.0 || >=20.0.0} + /@typescript-eslint/parser@8.2.0(eslint@8.57.0)(typescript@5.4.2): + resolution: {integrity: sha512-j3Di+o0lHgPrb7FxL3fdEy6LJ/j2NE8u+AP/5cQ9SKb+JLH6V6UHDqJ+e0hXBkHP1wn1YDFjYCS9LBQsZDlDEg==} + engines: {node: ^18.18.0 || ^20.9.0 || >=21.1.0} peerDependencies: - eslint: ^8.56.0 + eslint: ^8.57.0 || ^9.0.0 typescript: '*' peerDependenciesMeta: typescript: optional: true dependencies: - '@typescript-eslint/scope-manager': 7.18.0 - '@typescript-eslint/types': 7.18.0 - '@typescript-eslint/typescript-estree': 7.18.0(typescript@5.4.2) - '@typescript-eslint/visitor-keys': 7.18.0 + '@typescript-eslint/scope-manager': 8.2.0 + '@typescript-eslint/types': 8.2.0 + '@typescript-eslint/typescript-estree': 8.2.0(typescript@5.4.2) + '@typescript-eslint/visitor-keys': 8.2.0 debug: 4.3.4(supports-color@9.3.1) eslint: 8.57.0 typescript: 5.4.2 @@ -4082,12 +4082,12 @@ packages: '@typescript-eslint/visitor-keys': 5.62.0 dev: true - /@typescript-eslint/scope-manager@7.18.0: - resolution: {integrity: sha512-jjhdIE/FPF2B7Z1uzc6i3oWKbGcHb87Qw7AWj6jmEqNOfDFbJWtjt/XfwCpvNkpGWlcJaog5vTR+VV8+w9JflA==} - engines: {node: ^18.18.0 || >=20.0.0} + /@typescript-eslint/scope-manager@8.2.0: + resolution: {integrity: sha512-OFn80B38yD6WwpoHU2Tz/fTz7CgFqInllBoC3WP+/jLbTb4gGPTy9HBSTsbDWkMdN55XlVU0mMDYAtgvlUspGw==} + engines: {node: ^18.18.0 || ^20.9.0 || >=21.1.0} dependencies: - '@typescript-eslint/types': 7.18.0 - '@typescript-eslint/visitor-keys': 7.18.0 + '@typescript-eslint/types': 8.2.0 + '@typescript-eslint/visitor-keys': 8.2.0 dev: true /@typescript-eslint/type-utils@5.50.0(eslint@8.33.0)(typescript@4.9.4): @@ -4130,23 +4130,22 @@ packages: - supports-color dev: true - /@typescript-eslint/type-utils@7.18.0(eslint@8.57.0)(typescript@5.4.2): - resolution: {integrity: sha512-XL0FJXuCLaDuX2sYqZUUSOJ2sG5/i1AAze+axqmLnSkNEVMVYLF+cbwlB2w8D1tinFuSikHmFta+P+HOofrLeA==} - engines: {node: ^18.18.0 || >=20.0.0} + /@typescript-eslint/type-utils@8.2.0(eslint@8.57.0)(typescript@5.4.2): + resolution: {integrity: sha512-g1CfXGFMQdT5S+0PSO0fvGXUaiSkl73U1n9LTK5aRAFnPlJ8dLKkXr4AaLFvPedW8lVDoMgLLE3JN98ZZfsj0w==} + engines: {node: ^18.18.0 || ^20.9.0 || >=21.1.0} peerDependencies: - eslint: ^8.56.0 typescript: '*' peerDependenciesMeta: typescript: optional: true dependencies: - '@typescript-eslint/typescript-estree': 7.18.0(typescript@5.4.2) - '@typescript-eslint/utils': 7.18.0(eslint@8.57.0)(typescript@5.4.2) + '@typescript-eslint/typescript-estree': 8.2.0(typescript@5.4.2) + '@typescript-eslint/utils': 8.2.0(eslint@8.57.0)(typescript@5.4.2) debug: 4.3.4(supports-color@9.3.1) - eslint: 8.57.0 ts-api-utils: 1.3.0(typescript@5.4.2) typescript: 5.4.2 transitivePeerDependencies: + - eslint - supports-color dev: true @@ -4160,9 +4159,9 @@ packages: engines: {node: ^12.22.0 || ^14.17.0 || >=16.0.0} dev: true - /@typescript-eslint/types@7.18.0: - resolution: {integrity: sha512-iZqi+Ds1y4EDYUtlOOC+aUmxnE9xS/yCigkjA7XpTKV6nCBd3Hp/PRGGmdwnfkV2ThMyYldP1wRpm/id99spTQ==} - engines: {node: ^18.18.0 || >=20.0.0} + /@typescript-eslint/types@8.2.0: + resolution: {integrity: sha512-6a9QSK396YqmiBKPkJtxsgZZZVjYQ6wQ/TlI0C65z7vInaETuC6HAHD98AGLC8DyIPqHytvNuS8bBVvNLKyqvQ==} + engines: {node: ^18.18.0 || ^20.9.0 || >=21.1.0} dev: true /@typescript-eslint/typescript-estree@5.50.0(typescript@4.9.4): @@ -4207,17 +4206,17 @@ packages: - supports-color dev: true - /@typescript-eslint/typescript-estree@7.18.0(typescript@5.4.2): - resolution: {integrity: sha512-aP1v/BSPnnyhMHts8cf1qQ6Q1IFwwRvAQGRvBFkWlo3/lH29OXA3Pts+c10nxRxIBrDnoMqzhgdwVe5f2D6OzA==} - engines: {node: ^18.18.0 || >=20.0.0} + /@typescript-eslint/typescript-estree@8.2.0(typescript@5.4.2): + resolution: {integrity: sha512-kiG4EDUT4dImplOsbh47B1QnNmXSoUqOjWDvCJw/o8LgfD0yr7k2uy54D5Wm0j4t71Ge1NkynGhpWdS0dEIAUA==} + engines: {node: ^18.18.0 || ^20.9.0 || >=21.1.0} peerDependencies: typescript: '*' peerDependenciesMeta: typescript: optional: true dependencies: - '@typescript-eslint/types': 7.18.0 - '@typescript-eslint/visitor-keys': 7.18.0 + '@typescript-eslint/types': 8.2.0 + '@typescript-eslint/visitor-keys': 8.2.0 debug: 4.3.4(supports-color@9.3.1) globby: 11.1.0 is-glob: 4.0.3 @@ -4269,16 +4268,16 @@ packages: - typescript dev: true - /@typescript-eslint/utils@7.18.0(eslint@8.57.0)(typescript@5.4.2): - resolution: {integrity: sha512-kK0/rNa2j74XuHVcoCZxdFBMF+aq/vH83CXAOHieC+2Gis4mF8jJXT5eAfyD3K0sAxtPuwxaIOIOvhwzVDt/kw==} - engines: {node: ^18.18.0 || >=20.0.0} + /@typescript-eslint/utils@8.2.0(eslint@8.57.0)(typescript@5.4.2): + resolution: {integrity: sha512-O46eaYKDlV3TvAVDNcoDzd5N550ckSe8G4phko++OCSC1dYIb9LTc3HDGYdWqWIAT5qDUKphO6sd9RrpIJJPfg==} + engines: {node: ^18.18.0 || ^20.9.0 || >=21.1.0} peerDependencies: - eslint: ^8.56.0 + eslint: ^8.57.0 || ^9.0.0 dependencies: '@eslint-community/eslint-utils': 4.4.0(eslint@8.57.0) - '@typescript-eslint/scope-manager': 7.18.0 - '@typescript-eslint/types': 7.18.0 - '@typescript-eslint/typescript-estree': 7.18.0(typescript@5.4.2) + '@typescript-eslint/scope-manager': 8.2.0 + '@typescript-eslint/types': 8.2.0 + '@typescript-eslint/typescript-estree': 8.2.0(typescript@5.4.2) eslint: 8.57.0 transitivePeerDependencies: - supports-color @@ -4301,11 +4300,11 @@ packages: eslint-visitor-keys: 3.4.3 dev: true - /@typescript-eslint/visitor-keys@7.18.0: - resolution: {integrity: sha512-cDF0/Gf81QpY3xYyJKDV14Zwdmid5+uuENhjH2EqFaF0ni+yAyq/LzMaIJdhNJXZI7uLzwIlA+V7oWoyn6Curg==} - engines: {node: ^18.18.0 || >=20.0.0} + /@typescript-eslint/visitor-keys@8.2.0: + resolution: {integrity: sha512-sbgsPMW9yLvS7IhCi8IpuK1oBmtbWUNP+hBdwl/I9nzqVsszGnNGti5r9dUtF5RLivHUFFIdRvLiTsPhzSyJ3Q==} + engines: {node: ^18.18.0 || ^20.9.0 || >=21.1.0} dependencies: - '@typescript-eslint/types': 7.18.0 + '@typescript-eslint/types': 8.2.0 eslint-visitor-keys: 3.4.3 dev: true @@ -12199,23 +12198,6 @@ packages: - sugarss dev: true - /svelte-eslint-parser@0.36.0(svelte@5.0.0-next.208): - resolution: {integrity: sha512-/6YmUSr0FAVxW8dXNdIMydBnddPMHzaHirAZ7RrT21XYdgGGZMh0LQG6CZsvAFS4r2Y4ItUuCQc8TQ3urB30mQ==} - engines: {node: ^12.22.0 || ^14.17.0 || >=16.0.0} - peerDependencies: - svelte: ^3.37.0 || ^4.0.0 || ^5.0.0-next.115 - peerDependenciesMeta: - svelte: - optional: true - dependencies: - eslint-scope: 7.2.2 - eslint-visitor-keys: 3.4.3 - espree: 9.6.1 - postcss: 8.4.39 - postcss-scss: 4.0.9(postcss@8.4.39) - svelte: 5.0.0-next.208 - dev: true - /svelte-eslint-parser@0.41.0(svelte@5.0.0-next.208): resolution: {integrity: sha512-L6f4hOL+AbgfBIB52Z310pg1d2QjRqm7wy3kI1W6hhdhX5bvu7+f0R6w4ykp5HoDdzq+vGhIJmsisaiJDGmVfA==} engines: {node: ^12.22.0 || ^14.17.0 || >=16.0.0}