Skip to content

Conversation

@sabonerune
Copy link
Contributor

内容

e2eテストで使用するエンジンを更新します。
これによりLinuxのテスト時にインストールしていたlibsndfile1のインストールが不要になります。
またmacOSのテストで使うエンジンも変更します。

関連 Issue

ref VOICEVOX/voicevox_engine#1528
ref #2112

@sabonerune sabonerune requested a review from a team as a code owner June 25, 2025 15:59
@sabonerune sabonerune requested review from Hiroshiba and removed request for a team June 25, 2025 15:59
@voicevox-preview-pages
Copy link

voicevox-preview-pages bot commented Jun 25, 2025

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

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

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.

Pull Request Overview

Updates the e2e test workflow to use a newer, self-contained engine release and adjust platform asset names.

  • Bumps VOICEVOX_ENGINE_VERSION to 0.23.0
  • Removes libsndfile1 installation requirement
  • Updates matrix asset names for Linux and macOS
Comments suppressed due to low confidence (4)

.github/workflows/test.yml:12

  • Hyphen removed before VOICEVOX_ENGINE_VERSION; ensure the env block still conforms to GitHub Actions YAML mapping to avoid syntax errors.
  VOICEVOX_ENGINE_VERSION: "0.23.0"

.github/workflows/test.yml:71

  • [nitpick] Ensure 'linux-cpu-x64' matches the actual release artifacts and consider standardizing asset naming across platforms for clarity.
            voicevox_engine_asset_name: linux-cpu-x64

.github/workflows/test.yml:73

  • Switching macOS to only arm64 runners may exclude Intel-based coverage; consider re-adding an x64 matrix entry or making architecture selection configurable.
            voicevox_engine_asset_name: macos-arm64

.github/workflows/test.yml:85

  • Removed libsndfile installation; confirm the updated engine binary has no dependency on libsndfile, otherwise tests may fail due to missing library.
          sudo apt-get install -y xvfb x11-xserver-utils # for electron

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.

特に問題なさそう。マージします。

@sevenc-nanashi sevenc-nanashi enabled auto-merge June 27, 2025 01:18
@sevenc-nanashi sevenc-nanashi added this pull request to the merge queue Jun 27, 2025
Merged via the queue into VOICEVOX:main with commit 6aac48c Jun 27, 2025
11 checks passed
@sabonerune sabonerune deleted the fix/test-engine branch June 27, 2025 03:38
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