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

update poetry #278

Merged
merged 3 commits into from
Jun 8, 2024
Merged

update poetry #278

merged 3 commits into from
Jun 8, 2024

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
5 checks passed
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