-
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
engine-infos alias poc #2254
base: main
Are you sure you want to change the base?
engine-infos alias poc #2254
Conversation
TODO: voicevox/.github/workflows/test.yml Lines 135 to 141 in e49a2fc
|
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.
おー!!!!
なるほど〜〜〜 .d.ts
に型書けば良いのかぁ。なるほどです。
全テスト通すには結構いろいろ調整が必要かもです、不明な点あればなんでも聞いていただければ!!!
といってもvite周りは不慣れなので、わかるかわかりませんが・・・
いくつかこうできると嬉しいな〜〜〜って点を思いついたのでコメントさせていただきました!!!
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.
あ、.development
の方はcommitしないようにしてたりします! 🙇
(個々の環境で変更するものをgit管理すると大変なので)
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.
ちなみにenv全部まるまる移動することとかってできたりしますか・・・?
特に環境変数じゃないといけないような実装はしてないはずなのと、全部型付けられて幸せそうというのと・・・。
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.
もし可能ならrootディレクトリに直接.ts
ファイルを置けると嬉しい・・・!!
env.production.ts
みたいな感じとかできれば、管理しないといけないファイル数減らせるよな〜〜〜と。
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.
src/vite-env.d.ts
でも良いかも?
内容
VITE_DEFAULT_ENGINE_INFO
を環境変数からJSON文字列で指定していたが,これの型を起動時に検証したくないから,環境変数で指定するのではなく.ts
ファイルに書いて,どの.ts
を読むのかをvite.config.mts
で指定するようにした.その他
とりあえず
npm run electron:serve
だけは動くことを確認しました.他も追って確認します.