Skip to content

fix: args#41

Merged
tuna2134 merged 1 commit intovoicevox-client:mainfrom
Nakanishi123:main
Aug 17, 2023
Merged

fix: args#41
tuna2134 merged 1 commit intovoicevox-client:mainfrom
Nakanishi123:main

Conversation

@Nakanishi123
Copy link
Contributor

引数が間違っている

@tuna2134
Copy link
Member

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

コードの解説:

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

コードの改善点:

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

Copy link
Member

@tuna2134 tuna2134 left a comment

Choose a reason for hiding this comment

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

問題ないですね
LGTM

@tuna2134 tuna2134 merged commit 4f64fb9 into voicevox-client:main Aug 17, 2023
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