-
Notifications
You must be signed in to change notification settings - Fork 298
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
実装:MinimumEngineManifest格納時の検証をZodで行う #1186
Conversation
src/type/preload.ts
Outdated
}; | ||
|
||
export const minimumEngineManifest = z |
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.
ここもSchemaだと統一感あるかもです
export const minimumEngineManifest = z | |
export const minimumEngineManifestSchema = z |
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.
あ、ここ忘れられてるかもです!
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.
ほぼLGTMです!!
MinimumEngineManifestにしてるところって結構あったんですね。
ありがとうございます! |
src/type/preload.ts
Outdated
@@ -1,6 +1,7 @@ | |||
import { IpcRenderer, IpcRendererEvent, nativeTheme } from "electron"; | |||
import { IpcSOData } from "./ipc"; | |||
import { z } from "zod"; | |||
import { EngineManifest } from "@/openapi"; |
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.
ここ'EngineManifest' is defined but never used
と出てたので消しちゃっても良さそうです!
import { EngineManifest } from "@/openapi"; |
こちらのコメントどうでしょう 👀 (重要ではないのでそのままでも大丈夫です!) あとコンフリクトが発生していそうです 🙇♂️ |
* 命名規則の適用(スキーマ) * 不要な参照の削除
…nto Branch_4d3e408c
* EngineManageDialog.vue
途中までうまくいってたとおもったのですが |
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.
LGTM!!
VOICEVOXのgitログは全部squashされるので大丈夫です!!
問題ないと思うのでマージします!! |
内容
・エンジン追加時および起動時のminimumEngineManifestを Zodで検証してから格納する
・検証に当たり、portの型が異なるようにみえたので修正(string→number)
関連 Issue
ref #1181
スクリーンショット・動画など
その他