Skip to content

Conversation

@sevenc-nanashi
Copy link
Member

内容

mainのCIが落ちてるのを直します。

関連 Issue

(なし)

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

(なし)

その他

(なし)

@sevenc-nanashi sevenc-nanashi requested a review from a team as a code owner December 28, 2024 22:41
@sevenc-nanashi sevenc-nanashi requested review from Hiroshiba and removed request for a team December 28, 2024 22:41
} from "@/domain/hotkeyAction";

test("すべてのホットキーに初期値が設定されている", async () => {
const defaultHotkeySettings = getDefaultHotkeySettings(false);
Copy link
Member Author

Choose a reason for hiding this comment

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

このisMacの指定の部分、ぱっと見何やってるか判らないので{ isMac: boolean }を受け取るように死体気持ちがあります

Copy link
Member

Choose a reason for hiding this comment

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

ですね! ちょっとPR出します 🙏

@voicevox-preview-pages
Copy link

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

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

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

やっぱマージキューほしい!!!!!
(approve人数がレビュワー権限とメンテナ権限で違う関係でGithubの仕組みを使えない)

@Hiroshiba Hiroshiba merged commit 4cab1c5 into VOICEVOX:main Dec 29, 2024
10 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