-
Notifications
You must be signed in to change notification settings - Fork 198
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
MAINT: 未使用の依存パッケージを削除 #962
Conversation
atomicwrites = "^1.4.0" | ||
colorama = "^0.4.4" |
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.
ref: #219 (comment)
WSL上で作業したところcolorama
がWindowsのみインストールされるようにrequirements*.txt
が構成されていたため削除しても問題ないと思います。
(atomicwrites
は使用するパッケージが無くなったため削除)
soundfile = "^0.12.1" | ||
pyyaml = "^6.0" | ||
pyworld = "^0.3.0" | ||
requests = "^2.28.1" |
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.
@@ -48,23 +48,18 @@ numpy = "^1.20.0" | |||
fastapi = "^0.103.2" | |||
python-multipart = "^0.0.5" | |||
uvicorn = "^0.15.0" | |||
aiofiles = "^0.7.0" |
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.
不明
初期から入っているが使用している様子がないため削除
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.
これなんであるのかわからないですね・・・。ビルドに含めてたことがあったっぽい・・・?
Line 84 in fcb38c0
--follow-import-to=aiofiles \ |
jinja2 = "^3.1.2" | ||
pyopenjtalk = { git = "https://github.com/VOICEVOX/pyopenjtalk", rev = "b35fc89fe42948a28e33aed886ea145a51113f88" } | ||
semver = "^3.0.0" | ||
platformdirs = "^3.10.0" | ||
soxr = "^0.3.6" | ||
|
||
[tool.poetry.group.dev.dependencies] | ||
cython = "^0.29.34,>=0.29.33" # NOTE: for Python 3.11 |
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.
不要のはず。
(pyworld
が何故かinstall_requires
内で宣言しているため完全に削除不能)
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!!
まあ動かなくなったら動かなくなったということでrevertすれば良いかなと!
ビルド後のテストも回したいので0.15.0-preview.11
としてrelease作成してみます。
内容
pyploject.toml
に記載されているが現在使用されていないパッケージを削除します。