-
Notifications
You must be signed in to change notification settings - Fork 196
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
非推奨化されたdistutils.versionへの依存をなくし、python-semverに移行する #609
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ありがとうございます!!
いくつかコメントしてみましたが、方針は賛成です!
@aoirint こちらどうでしょう 👀 |
遅くなりましたが返答しました! |
移行先を
|
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!!
PRありがとうございます!
こうやってちょっとずつ綺麗にしていくの、大事なのですがなかなか手が回っていないのでとても嬉しいです・・・!
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.
1点コメントしましたが、LGTMです。
Poetryのlockファイルがコンフリクトしているみたいなので、解消の方をお願いします:bow:
解消しました。
|
ありがとうございます! たぶん大丈夫だと思うのでマージします!! |
内容
PEP 632で非推奨化された、distutilsへの依存をなくします。
コアのmetasからバージョンをパースし、最新のコアを選択するために
distutils.version.LooseVersion
が使われていました。代替として、(切り替え先をpackaging.version.parse
に切り替えました。python-semver
に変更)代替として、python-semverに切り替えました。
また、最新コアの選択ロジックを
utility.core_version
に切り出し、テストを追加しました(他に追加した方がよさそうなテストケース/あり得るコアのバージョンがあれば教えてください)。(切り替え先をpackaging.version.parse
の戻り値の型がpackaging.version.Version | packaging.version.LegacyVersion
になっていて、packaging.version.LegacyVersion
が含まれるsetuptools
/packaging
のバージョンがあるかもしれませんが、削除されていると判断して
packaging.version.Version
のみとみなしています(このあたりのパッケージのバージョン管理がどうなっているかよくわからないです...)。python-semver
に変更)ref: Remove support for LegacyVersion pypa/setuptools#2497ref: DropLegacySpecifier
andLegacyVersion
pypa/packaging#407関連 Issue
スクリーンショット・動画など
その他