Skip to content

chore: 一部のテストのタイムアウトを長くする + vitestの警告を直す - #2703

Merged
sevenc-nanashi merged 2 commits into
VOICEVOX:mainfrom
sevenc-nanashi:chore/longer-test-timeout
Jul 9, 2025
Merged

chore: 一部のテストのタイムアウトを長くする + vitestの警告を直す#2703
sevenc-nanashi merged 2 commits into
VOICEVOX:mainfrom
sevenc-nanashi:chore/longer-test-timeout

Conversation

@sevenc-nanashi

Copy link
Copy Markdown
Member

内容

"再生ヘッド位置に近いフレーズから優先的にレンダリングされる"が頻繁に落ちてたので延長します。

関連 Issue

(なし)

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

(なし)

その他

(なし)

@sevenc-nanashi
sevenc-nanashi requested a review from a team as a code owner July 7, 2025 23:02
@sevenc-nanashi
sevenc-nanashi requested review from Hiroshiba and removed request for a team July 7, 2025 23:02
@voicevox-preview-pages

voicevox-preview-pages Bot commented Jul 7, 2025

Copy link
Copy Markdown

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

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

@Hiroshiba

Hiroshiba commented Jul 8, 2025

Copy link
Copy Markdown
Member

テスト落ちてそうなのと、タイトルに関係ない部分のファイル変更が大きそう?
(後者はこれもついでにということなのか、間違えて混入しているのかどっちでしょうか。テスト落ちてる原因もそこにありそう?)

@sevenc-nanashi

Copy link
Copy Markdown
Member Author

あ、vitest.workspace.tsは別のprにいれる予定のやつが混入したやつですね。

とはいっても一緒にやって良さそうではある。

@Hiroshiba

Copy link
Copy Markdown
Member

別に一緒でもダメではない気もしました。
でもまあ大部分の変更がそのついでの方なので、逆にそっちをPRのタイトルにしておかないと後でわかんなくなりそうな気がしました!

@Hiroshiba Hiroshiba left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM!!

タイトルの変更か、PR分けだけやっちゃっていただければマージで良さそう!

@sevenc-nanashi sevenc-nanashi changed the title chore: 一部のテストのタイムアウトを長くする chore: 一部のテストのタイムアウトを長くする + vitestの警告を直す Jul 9, 2025
@sevenc-nanashi

Copy link
Copy Markdown
Member Author

マージします。

@sevenc-nanashi
sevenc-nanashi enabled auto-merge July 9, 2025 00:00
@sevenc-nanashi
sevenc-nanashi added this pull request to the merge queue Jul 9, 2025
Merged via the queue into VOICEVOX:main with commit 576fae3 Jul 9, 2025
11 checks passed
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