Skip to content
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

ViteでビルドをしながらVSCodeでDebugを行えるようにする #1221

Merged
merged 13 commits into from
Mar 1, 2023

Conversation

yamachu
Copy link
Member

@yamachu yamachu commented Feb 21, 2023

内容

#1207
で vue-cli から vite にビルド方法が変更になった
初期状態だとソースコードと対応させたデバッグを行うための sourcemap を出力しないようになっているため、serve 系の development 環境では出力するように変更する

関連 Issue

スクリーンショット・動画など

その他

port番号は適当なので、お好みのがあれば変えたい
また .vscode/launch.json は gitignore しているため、 launch.template.json とした
実際に使用する際は自分で launch.json にリネームして使うこと

@yamachu yamachu requested a review from a team as a code owner February 21, 2023 15:10
@yamachu yamachu requested review from Hiroshiba and removed request for a team February 21, 2023 15:10
vite.config.ts Outdated Show resolved Hide resolved
Co-authored-by: Nanashi. <sevenc_nanashi@yahoo.co.jp>
yamachu and others added 2 commits February 24, 2023 00:53
Co-authored-by: Hiroshiba <hihokaruta@gmail.com>
src/background.ts Outdated Show resolved Hide resolved
Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

launch.template.jsonについて、READMEに書くのが良いのかなと思いました!

  • .vscodeはignoreしてる人も多そう
  • VOICEVOXとしてサポートし続けることは難しい
  • でもElectronに慣れてない方にはたしかに便利

ということで、READMEで案内しつつ、あまり目立たないようにし、Vite+Electron環境を想定していることもどこかに書いておけばデグレってることに気づけるかなと。

まあでもこれはドキュメント整理的なタスクをお願いしてることになりそうです。
もし面倒であれば、とりあえずこのjsonファイルを消していただければ、僕の方でREADMEに追加しておこうと思います

@Hiroshiba
Copy link
Member

あ! あとコンフリクトが起こっていそうでした 🙇‍♂️
(diff見るとややこしいですが、sourcemapキーが増えただけっぽみ)

Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!!!

ちなみに僕の環境だとlaunchは動きませんでした 😇
詳しいエラーが全く見えないのでなぜかはわかりませんでした。
デバッガーが必要になったら再挑戦してみます。

@Hiroshiba
Copy link
Member

コンフリクト発生していたので解消してみました。

@y-chan .vscodeディレクトリをメンテすることに関して、どうでしょう・・・?
このディレクトリはgitignoreされてる方多そうなでなんか変な挙動しそうだなとか思っています。
どこか違うとこに移動した方が良いですかね・・・・・・?
<details>で囲ってREADMEにとか・・・?)

vite.config.ts Outdated

rmSync(path.resolve(__dirname, "dist"), { recursive: true, force: true });

const isElectron = process.env.VITE_IS_ELECTRON === "true";

export default defineConfig((options) => {
const isDevelopment = options.mode === "development";
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

テスト時もオンにしても良さそう?(テスト環境だとmodeが"test"になる)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

テスト環境は本番に近い内容のが良いという考え方はあるかも。でもsource mapは別かも。
Webフロントめちゃ詳しそうな @yamachu さん的にはどうでしょ 👀

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sourcemapの出力ぐらいで特に挙動が変わるみたいのはないので、e2eテストとかをしている間にどうしてもわからん壊れ方してるのを調査したいって時には役に立つかもですね
productionで出なければいいので、変数名を変えてテスト環境でも出していいかも

Co-authored-by: Nanashi. <sevenc_nanashi@yahoo.co.jp>
@y-chan
Copy link
Member

y-chan commented Mar 1, 2023

.vscodeディレクトリをメンテする
このディレクトリはgitignoreされてる方多そう

確かにそうですが、launth.template.jsonがある場合はないためにコンフリクトは起きないはずなので大丈夫な気がします。
他のプロジェクトでも、.vscodesetting.jsonなどを保守する場合がそこそこあるようなので、特に懸念はないかなという気がしています。

@Hiroshiba
Copy link
Member

良い気がしますね!!
たぶんapproveだと思う&テスト通ってるのでマージします!

@Hiroshiba Hiroshiba merged commit 9b2a7ec into VOICEVOX:main Mar 1, 2023
@yamachu yamachu deleted the debuggable branch June 21, 2023 13:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants