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

add: poetry バージョン固定 #1075

Closed
3 tasks done
tarepan opened this issue Feb 26, 2024 · 2 comments · Fixed by #1079
Closed
3 tasks done

add: poetry バージョン固定 #1075

tarepan opened this issue Feb 26, 2024 · 2 comments · Fixed by #1079

Comments

@tarepan
Copy link
Contributor

tarepan commented Feb 26, 2024

内容

提案概要: poetry バージョンの厳密な固定

現在の VOICEVOX ENGINE は poetry のバージョンを緩く管理している:

poetry = "^1.3.1"

一方で lock ファイルでは厳密に 1.6.1 を要求している:

voicevox_engine/poetry.lock

Lines 1411 to 1413 in 0c698d6

[[package]]
name = "poetry"
version = "1.6.1"

poetry==1.6.1 ; python_version >= "3.11" and python_version < "3.12"

そして lock ファイル側は poetry バージョンに依存した文字列が存在しており、異なるバージョンを利用するとここに(無意味な)diffが発生する:

# This file is automatically @generated by Poetry 1.6.1 and should not be changed by hand.

lock ファイル形式がpoetry バージョンに依存するという危険性は以前より指摘されており(#750)、緩いバージョン指定はこの危険性を大きくする。

このような背景から、poetry バージョン固定を提案します。

Pros 良くなる点

  • 無意味 diff の予防
  • 良きせぬフォーマット変更の予防

Cons 悪くなる点

  • global 環境で多数のプロジェクトを並列開発しづらくなる

実現方法

  • pyproject.toml バージョン指定の厳密化

VOICEVOXのバージョン

0.16.0

OSの種類/ディストリ/バージョン

  • Windows
  • macOS
  • Linux
@github-actions github-actions bot added OS 依存:linux Linux に依存した現象 OS 依存:mac macOS に依存した現象 OS 依存:win Windows に依存した現象 labels Feb 26, 2024
@Hiroshiba
Copy link
Member

勘違いしそうなので厳密な固定が良いと思いました!

まあたぶん全部厳密に固定した方が良い気もします。
まあ衝突した時大変ですが…。

@tarepan
Copy link
Contributor Author

tarepan commented Feb 26, 2024

着手します。

@tarepan tarepan removed OS 依存:mac macOS に依存した現象 OS 依存:linux Linux に依存した現象 OS 依存:win Windows に依存した現象 labels Mar 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants