Skip to content
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

Migrate: Poetryに移行 #652

Merged
merged 12 commits into from
Oct 23, 2023
Merged

Conversation

sevenc-nanashi
Copy link
Member

@sevenc-nanashi sevenc-nanashi commented Oct 21, 2023

内容

requirements-*.txtを爆破してPoetryのDependency Groupに移行します。

メインの依存はメジャーバージョン範囲内、開発依存は固定しています。

関連 Issue

その他

(なし)

@sevenc-nanashi sevenc-nanashi marked this pull request as draft October 21, 2023 14:13
Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

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

完成してるのかわからないけど、LGTM!!

.github/workflows/test.yml Outdated Show resolved Hide resolved
.github/workflows/python_lint.yml Outdated Show resolved Hide resolved
@@ -2,6 +2,18 @@
name = "voicevox_core"
requires-python = ">=3.8"
dependencies = ["numpy", "pydantic>=1.9.2,<2"]
description = "VOICEVOX CORE の Python バインディングです。"
authors = [{ name = "Hiroshiba", email = "hihokaruta@gmail.com" }]
Copy link
Member

Choose a reason for hiding this comment

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

これそのうちvoicevox用のに一斉に変えたいですね!
とりあえずまだ決まってないのでこれで。

@sevenc-nanashi sevenc-nanashi marked this pull request as ready for review October 22, 2023 00:00
.github/workflows/test.yml Outdated Show resolved Hide resolved
Co-authored-by: Ryo Yamashita <qryxip@gmail.com>
@qryxip
Copy link
Member

qryxip commented Oct 22, 2023

@sevenc-nanashi 以下の2つが無言resolveになっている件、これは単純な疑問なのですが、そちらに未pushのコミットがあったりしませんか?

@sevenc-nanashi
Copy link
Member Author

ありました!ごめんなさい

"Operating System :: POSIX :: Linux",
"Operating System :: Microsoft :: Windows",
"Operating System :: MacOS :: MacOS X",
"Environment :: GPU :: NVIDIA CUDA",
Copy link
Member

Choose a reason for hiding this comment

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

私は詳しくないのですが、この記述はどちらかというと"CUDA"が必要という方向を意味したりしないでしょうか?
例えばonnxruntime-gpuとかではこの指定は無いようです。

Copy link
Member

Choose a reason for hiding this comment

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

なんかタグっぽみ。・・・個人的にはまあもう完全にどっちでもいいかなと思いました!!
https://pypi.org/classifiers/

Copy link
Member Author

Choose a reason for hiding this comment

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

まぁそのままで良いと思います。

Copy link
Member

@qryxip qryxip left a comment

Choose a reason for hiding this comment

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

LGTM!

@qryxip qryxip merged commit bb08de7 into VOICEVOX:main Oct 23, 2023
32 checks passed
@qryxip qryxip mentioned this pull request Oct 25, 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.

3 participants