-
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
整理: dict
モジュールへの移植
#1061
整理: dict
モジュールへの移植
#1061
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.
PRありがとうございます!!
Pythonのビルトイン関数のdict
と混じってちょっとメンテナンス性下がるかもと感じました!
user_dict
はどうでしょう?
👍
👍 @Hiroshiba |
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!!
内容
概要: ユーザー辞書関連モジュールを新しい
dict
モジュール傘下へ移植してリファクタリング現在のユーザー辞書機能は2つのモジュールによって実装されている。
これら2モジュールは
voicevox_engine
直下に置かれており、かつ、ファイル名からは関連性が読み取れない。これらを新設の
dict
モジュール傘下へ移設すれば、より直感的なディレクトリ構造となる。このような背景から、ユーザー辞書関連モジュールを新しい
dict
モジュール傘下へ移植してリファクタリングすることを提案します。関連 Issue
無し