-
Notifications
You must be signed in to change notification settings - Fork 2
[Dressca] TypeScript 5.8.3 にアップデートする #2662
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
The head ref may contain hidden characters: "feaure/(Dressca)TS5.8.3\u306B\u30A2\u30C3\u30D7\u30C7\u30FC\u30C8"
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
このPRは、TypeScriptのバージョンを5.8.3にアップグレードし、create‑vue 3.17.0に追従するためにtsconfigの設定を更新するものです。
- TypeScriptのバージョンおよび関連パッケージの更新
- tsconfigファイルにおける"extends"の更新と不要な"composite"プロパティの削除
- cypressなどの設定ファイルにおけるinclude/excludeの見直し
Reviewed Changes
Copilot reviewed 10 out of 11 changed files in this pull request and generated no comments.
Show a summary per file
File | Description |
---|---|
samples/web-csr/dressca-frontend/consumer/tsconfig.vitest.json | "composite"プロパティの削除とテスト対象ファイルのinclude追加 |
samples/web-csr/dressca-frontend/consumer/tsconfig.node.json | "extends"の変更と"composite"プロパティの削除 |
samples/web-csr/dressca-frontend/consumer/tsconfig.app.json | "composite"と"baseUrl"の削除、pathsの維持 |
samples/web-csr/dressca-frontend/consumer/package.json | type-checkコマンドと依存するTypeScriptのバージョン更新 |
samples/web-csr/dressca-frontend/consumer/cypress/tsconfig.json | include/exclude設定の更新 |
samples/web-csr/dressca-frontend/admin/tsconfig.vitest.json | "composite"プロパティの削除とテスト対象ファイルのinclude追加 |
samples/web-csr/dressca-frontend/admin/tsconfig.node.json | "extends"の変更と"composite"プロパティの削除 |
samples/web-csr/dressca-frontend/admin/tsconfig.app.json | "composite"および"baseUrl"の削除、pathsの維持 |
samples/web-csr/dressca-frontend/admin/package.json | type-checkコマンドと依存パッケージのTypeScriptバージョン更新 |
samples/web-csr/dressca-frontend/admin/cypress/tsconfig.json | include/exclude設定の更新 |
Files not reviewed (1)
- samples/web-csr/dressca-frontend/package-lock.json: Language not supported
Comments suppressed due to low confidence (8)
samples/web-csr/dressca-frontend/consumer/tsconfig.vitest.json:5
- tsconfigファイルからcompositeプロパティを削除していますが、プロジェクト参照を利用している場合は、削除による影響を十分検証してください。
"compilerOptions": {
samples/web-csr/dressca-frontend/consumer/tsconfig.node.json:4
- compositeプロパティの削除が行われています。プロジェクトリファレンスを使用していないことを再確認してください。
"compilerOptions": {
samples/web-csr/dressca-frontend/consumer/tsconfig.app.json:5
- compositeプロパティが削除されています。プロジェクト内のbuild設定に影響しないか確認をお願いします。
"compilerOptions": {
samples/web-csr/dressca-frontend/admin/tsconfig.app.json:11
- adminのアプリ用tsconfigからcompositeプロパティが削除されています。buildやプロジェクト参照への影響を再度ご確認ください。
"compilerOptions": {
samples/web-csr/dressca-frontend/consumer/package.json:16
- [nitpick] 以前のコマンドでは--forceオプションが使用されていましたが、今回削除されています。意図的な変更であれば問題ありませんが、型エラーによるビルド失敗が懸念される場合は再検討してください。
"type-check": "vue-tsc --build",
samples/web-csr/dressca-frontend/admin/package.json:16
- [nitpick] 以前は--forceオプション付きでしたが、今回削除されています。型エラーが原因でビルドが止まる可能性があるため、意図に合わせた運用かどうか確認が必要です。
"type-check": "vue-tsc --build",
この Pull request で実施したこと
-TypeScript 5.8.3 にアップグレードしました。
-
npm list -all typescript
を実行し、すべてのパッケージがTypescript 5.8.3 を使用していることを確認しました。-あわせてTypeScritp の設定ファイル tsconfig について、create-vue 3.17.0 に追従するよう変更しました。
この Pull request では実施していないこと
ドキュメントの更新は別PRで実施予定です。
Issues や Discussions 、関連する Web サイトなどへのリンク