Skip to content

Commit 436c210

Browse files
committed
pr review fixes: upgrade versions, change linting/formatting/ts configs
1 parent a0043c6 commit 436c210

File tree

8 files changed

+1837
-764
lines changed

8 files changed

+1837
-764
lines changed

package-lock.json

Lines changed: 1811 additions & 750 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -17,12 +17,12 @@
1717
},
1818
"devDependencies": {
1919
"@eslint/eslintrc": "^3",
20-
"@types/node": "^20",
20+
"@types/node": "^22",
2121
"@types/react": "^18.0.28",
2222
"@types/react-dom": "^18.0.11",
23-
"@typescript-eslint/eslint-plugin": "^5.43.0",
24-
"@typescript-eslint/parser": "^5.43.0",
25-
"eslint": "^8.27.0",
23+
"@typescript-eslint/eslint-plugin": "^8.33.1",
24+
"@typescript-eslint/parser": "^8.33.1",
25+
"eslint": "^9.0.0",
2626
"eslint-config-next": "15.3.2",
2727
"eslint-config-prettier": "^8.5.0",
2828
"eslint-plugin-import": "^2.29.1",
@@ -40,5 +40,8 @@
4040
"*.ts": "eslint --cache --fix",
4141
"*.jsx": "eslint --cache --fix",
4242
"*.tsx": "eslint --cache --fix"
43+
},
44+
"engines": {
45+
"node": ">=22"
4346
}
4447
}

src/ui/.eslintignore

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

src/ui/.eslintrc.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,5 +44,6 @@
4444
],
4545
"import/no-extraneous-dependencies": ["error"],
4646
"no-secrets/no-secrets": ["error", { "additionalRegexes": {}, "ignoreContent": [] }]
47-
}
47+
},
48+
"settings": { "next": { "rootDir": ["src/ui/"] } }
4849
}

src/ui/.prettierignore

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,3 @@
1-
/node_modules
2-
.vscode
31
/bin
42
/build
53
/public

src/ui/.prettierrc

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,8 @@
22
"arrowParens": "always",
33
"bracketSameLine": false,
44
"bracketSpacing": true,
5-
"jsxBracketSameLine": false,
65
"jsxSingleQuote": false,
7-
"printWidth": 100,
6+
"printWidth": 88,
87
"semi": true,
98
"singleQuote": true,
109
"tabWidth": 2,

src/ui/app/page.tsx

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,13 @@ export default function Home() {
6262
target="_blank"
6363
rel="noopener noreferrer"
6464
>
65-
<Image aria-hidden src="/window.svg" alt="Window icon" width={16} height={16} />
65+
<Image
66+
aria-hidden
67+
src="/window.svg"
68+
alt="Window icon"
69+
width={16}
70+
height={16}
71+
/>
6672
Examples
6773
</a>
6874
<a

src/ui/tsconfig.json

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"compilerOptions": {
3-
"allowJs": true,
3+
"allowJs": false,
44
"allowSyntheticDefaultImports": true,
55
"baseUrl": "src/ui",
66
"esModuleInterop": true,
@@ -26,6 +26,13 @@
2626
"strict": true,
2727
"target": "ES2022"
2828
},
29-
"include": ["**/*.ts", "**/*.tsx", ".next/types/**/*.ts", "next-env.d.ts", "src", "app"],
29+
"include": [
30+
"**/*.ts",
31+
"**/*.tsx",
32+
".next/types/**/*.ts",
33+
"next-env.d.ts",
34+
"src",
35+
"app"
36+
],
3037
"exclude": [".meta/**/*", ".next/**/*", ".vscode", "node_modules"]
3138
}

0 commit comments

Comments
 (0)