-
Notifications
You must be signed in to change notification settings - Fork 205
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.
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
整理: 返り値型 #1078
整理: 返り値型 #1078
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
返り値の型書くのまあまあ面倒なのですが、まあ恩恵のが大きいかなぁと思いました。
ちょっと意見募りたいかもなので、メンション失礼します!
@PickledChair @takana-v @y-chan
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
返り値の型をしっかり書くのに賛成です! 基本はその方針で行って、何か問題が生じたら別途対応するのが良いと思いました |
内容
現在の VOICEVOX ENGINE は返り値の型付け(例:
-> None
)を省略しているコードが多数存在する。mypy の strict モードではこれらがエラー判定されるため、より安全な型チェックの阻害因子となっている。
このような背景から、返り値の型付けによるリファクタリングを提案します。
関連 Issue
無し