Skip to content

Conversation

@sevenc-nanashi
Copy link
Member

内容

タイトル通りです。

関連 Issue

(なし)

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

(なし)

その他

(なし)

@sevenc-nanashi sevenc-nanashi requested a review from a team as a code owner June 30, 2025 15:14
@sevenc-nanashi sevenc-nanashi requested review from Hiroshiba and removed request for a team June 30, 2025 15:14
@voicevox-preview-pages
Copy link

voicevox-preview-pages bot commented Jun 30, 2025

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

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

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.

@sevenc-nanashi こちらちょっと文脈が足りていないので、足していただければ!!

  • そもそもどういう方針にするのか
  • それは誰からの合意が得られているのか
  • なぜそれで問題がないのか
  • どういう利点があるのか

@sevenc-nanashi
Copy link
Member Author

生放送より:

そもそもどういう方針にするのか

  • マイナーバージョン1個分は様子見し、特にバグがなければ常にオンにする。

それは誰からの合意が得られているのか

  • 生放送で @Hiroshiba さんと話しました。

なぜそれで問題がないのか

どういう利点があるのか

  • 複数選択に気がついてないユーザーが複数選択を使える。

@Hiroshiba
Copy link
Member

Hiroshiba commented Jun 30, 2025

メモ完璧だと思います、ありがとうございます!!

これでPR見る人が「実験的機能から情報を引き継いでないのはバグではない」とわかるし、
なぜこの方針を取ったのか僕達が見返したときや、他の人が見たときに聞かなくてもわかるはず。

あ、 マイナーバージョン1個分は様子見しの理由がどういう利点があるのかに書かれてないかもですね!
書いときます:

  • 致命的なバグがあったときにユーザー側がオフにすることで対処できてUXが少しマシになるから
  • すでに実装済みの分岐をそのまま使い回すので工数はほとんどかからないため

@Hiroshiba
Copy link
Member

(まだテスト落ちてそう?)

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!!

ポストしようか迷いましたが、一度ポストしてそうだったので別に良い・・・かな!
https://x.com/voicevox_pj/status/1737282952311238896

(2年前・・・!)

@Hiroshiba Hiroshiba enabled auto-merge July 2, 2025 10:57
@Hiroshiba Hiroshiba disabled auto-merge July 2, 2025 10:57
@Hiroshiba Hiroshiba changed the title feat(複数選択): 複数選択を実験的機能から外す feat(複数選択): 複数選択を実験的機能から正式機能にする Jul 2, 2025
@Hiroshiba Hiroshiba enabled auto-merge July 2, 2025 10:58
@Hiroshiba Hiroshiba added this pull request to the merge queue Jul 2, 2025
Merged via the queue into VOICEVOX:main with commit 507f9bd Jul 2, 2025
12 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