-
Notifications
You must be signed in to change notification settings - Fork 309
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
Change: zodを使うように #1078
Change: zodを使うように #1078
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.
見てみました!!
仕様を把握するところから含めてかなり大変な変更だったと思います、ありがとうございます!!
HotkeySettings
など、コードが移動しているだけだったりarray→enumに変わっている箇所は、完全一致していたり数が一致していたりしたので、ちゃんとdiffはなさそうでした!
ちょっと仕様がわからない点がいくつかあったのでコメントしてみました!
} | ||
import { z } from "zod"; | ||
|
||
export const isMac = process.platform === "darwin"; |
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.
default: [], | ||
}, | ||
}, | ||
schema: electronStoreJsonSchema.properties as Schema<ElectronStoreType>, |
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.
ここのas
を取るいい方法考えてみたのですが思いつきませんでした 😇
zodToJsonSchema
が型消しちゃうんですねぇ
Co-authored-by: Hiroshiba <[email protected]>
Co-authored-by: Hiroshiba <[email protected]>
多分大丈夫だと思うのと、時間が経つほどコンフリクトが発生しやすい分野だと思うので、マージしちゃいます!! |
内容
タイトル通り。
関連 Issue
スクリーンショット・動画など
(なし)
その他
サンプリングレートをdefaultからengineDefaultにしないと動かないと思います。(#1056のブランチから始めるべきだった…)