Skip to content

Conversation

@Hiroshiba
Copy link
Member

@Hiroshiba Hiroshiba commented Jan 3, 2025

内容

今までelectron:e2eテストをするとき、test.yml内で.env.test.envにcpしてから書き換えるということをしていましたが、専用の.env.test-e2eを作ってそこから編集するように変更しました。
ブラウザe2eテストをモックエンジンで行うPRで.env.testを使おうと思っている中、この処理コピー&編集処理が厄介だったので独立にしました。

関連 Issue

から切り出しました。

その他

@Hiroshiba Hiroshiba requested a review from a team as a code owner January 3, 2025 11:45
@Hiroshiba Hiroshiba requested review from Copilot and removed request for a team January 3, 2025 11:46
@Hiroshiba Hiroshiba marked this pull request as draft January 3, 2025 11:46
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

Files not reviewed (1)
  • .env.test-electron: Language not supported

@voicevox-preview-pages
Copy link

voicevox-preview-pages bot commented Jan 3, 2025

🚀 プレビュー用ページを作成しました 🚀

更新時点でのコミットハッシュ:85968ce

@Hiroshiba Hiroshiba force-pushed the electron用の.envを作成 branch from d1ed175 to be35707 Compare January 3, 2025 12:01
@Hiroshiba Hiroshiba marked this pull request as ready for review January 3, 2025 12:02
@Hiroshiba
Copy link
Member Author

.envファイルの扱い方をどうしようか迷っていて、知見ありそうな @sevenc-nanashi さんにレビューリクエストしました 🙇

ぶっちゃけどう管理するのが良いかよくわからない・・・。
tests/以下に.envがいっぱい入ってるディレクトリを作る・・・?

@sevenc-nanashi
Copy link
Member

tests/以下に.envがいっぱい入ってるディレクトリを作る・・・?

これが安定かな~って思います。

@Hiroshiba
Copy link
Member Author

このPRで.env系が結構増えるので、

  • .env.test-e2e
  • .env.test-electron-default-vvpp

だけディレクトリ移動してみます!

image

@Hiroshiba
Copy link
Member Author

Hiroshiba commented Jan 3, 2025

@sevenc-nanashi こんな感じでどうでしょう・・・?
tests/env/以下に移動)

Copy link
Member

@sevenc-nanashi sevenc-nanashi left a comment

Choose a reason for hiding this comment

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

よさそう!

@Hiroshiba
Copy link
Member Author

レビューありがとうございます!!! マージします!

@Hiroshiba Hiroshiba enabled auto-merge January 3, 2025 13:52
@Hiroshiba Hiroshiba added this pull request to the merge queue Jan 3, 2025
Merged via the queue into VOICEVOX:main with commit faf91f0 Jan 3, 2025
11 checks passed
@Hiroshiba Hiroshiba deleted the electron用の.envを作成 branch January 3, 2025 14:01
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.

2 participants