New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix: args #41

Merged

tuna2134 merged 1 commit into voicevox-client:main from Nakanishi123:main

Aug 17, 2023

Contributor

Nakanishi123 commented Aug 17, 2023

引数が間違っている


          fix: args

dba6244

Member

tuna2134 commented Aug 17, 2023

🤖 このメッセージはAutoCodeReviewによって行われたコードレビューです 🤖

コードの解説：

create_audio_query_from_preset関数に渡される引数が間違っているため修正が必要です。
AudioQueryオブジェクトの初期化時に不要な引数が渡されているため修正が必要です。

コードの改善点：

core_versionが間違った引数で渡された場合にエラーメッセージを表示するように修正してください。
AudioQueryオブジェクトの初期化時に不要な引数preset_idを削除してください。

tuna2134 approved these changes

View reviewed changes

Member

tuna2134 left a comment

問題ないですね
LGTM

tuna2134 merged commit 4f64fb9 into voicevox-client:main

1 check passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet