Skip to content

feat(複数選択): 複数選択を実験的機能から正式機能にする - #2697

Merged
Hiroshiba merged 9 commits into
VOICEVOX:mainfrom
sevenc-nanashi:feat/stablize-multi-select
Jul 2, 2025
Merged

feat(複数選択): 複数選択を実験的機能から正式機能にする#2697
Hiroshiba merged 9 commits into
VOICEVOX:mainfrom
sevenc-nanashi:feat/stablize-multi-select

Conversation

@sevenc-nanashi

Copy link
Copy Markdown
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

voicevox-preview-pages Bot commented Jun 30, 2025

Copy link
Copy Markdown

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

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

@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.

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

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

@sevenc-nanashi

Copy link
Copy Markdown
Member Author

生放送より:

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

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

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

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

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

どういう利点があるのか

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

@Hiroshiba

Hiroshiba commented Jun 30, 2025

Copy link
Copy Markdown
Member

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

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

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

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

@Hiroshiba

Copy link
Copy Markdown
Member

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

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

ポストしようか迷いましたが、一度ポストしてそうだったので別に良い・・・かな!
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
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