Skip to content

Conversation

@mo124121
Copy link
Contributor

Why is this change needed?

  • 開発環境セットアップ時のトラブルを回避するため。
  • 今後の依存関係更新を行うときにpoetryの更新によるものとの差分を区別するため。

コントリビューションガイドのセットアップ方法の記載が古く利用ができない。
変更前ではpoetryの日本語訳を行っている有志サイトを参照しているが、そちらの更新が止まっている。
https://cocoatomo.github.io/poetry-ja/
一方で公式の方は上記記事記載の方法をdeprecateとしている。
python-poetry/poetry#6377

また、poetry 1.1以前と1.2以降でlockファイルのフォーマットが変更になっている。
https://github.com/orgs/python-poetry/discussions/6763
python-poetry/poetry#5702

マージ先のlockファイル生成時のpoetryバージョンは不明だが、フォーマット的に1.1以前とみられる。
新しいpoetryを用いてlockを行うと差分が大きくなる。

What did you implement?

  • (英語ではあるが)公式のドキュメントを参照するようにした。
  • 現時点で最新のpoetry 1.6.1を用いたlockファイルの再生成を行った。

What behavior do you expect?

インストールされるパッケージに変更前後で差分がないこと。
手元環境ではコミット前後においてインストールされるpipパッケージリストに変更がないことを確認。

  • WSL2 ubuntu22.04
  • CPython3.11.4

other

OSSへのコントリビュートが初めてなので、作法がわかっていません。
大変申し訳ないのですが、おかしい部分があれば適宜ご指摘いただけると助かります。

@kyuridenamida
Copy link
Owner

丁寧な背景の説明ありがとうございます。勉強になりました。
poetry.lockが最新のstableに対してconflictを起こしていたのでこちらで直しました。マージさせていただきます。

@kyuridenamida kyuridenamida modified the milestones: 2.15.0, 2.14.1 Jun 8, 2024
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.18%. Comparing base (30336a7) to head (7d64497).
Report is 12 commits behind head on stable.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@ Coverage Diff @@ ## stable #278 +/- ## ========================================== - Coverage 90.31% 90.18% -0.14%  ========================================== Files 64 64 Lines 2788 2751 -37 ========================================== - Hits 2518 2481 -37  Misses 270 270 

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@kyuridenamida kyuridenamida merged commit 79dc55d into kyuridenamida:stable Jun 8, 2024
@kyuridenamida kyuridenamida modified the milestones: 2.14.1, 2.15.0 Nov 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants