Skip to content

Conversation

@cdce8p
Copy link
Contributor

@cdce8p cdce8p commented Sep 23, 2025

Update poetry-core to >=2.2.0 and move basic config from tool.poetry to project table.

Metadata diff

 ...
-License: Apache-2.0
+License-Expression: Apache-2.0
 License-File: LICENSE.txt
 ...
-Requires-Python: >=3.9.2,<4.0
+Requires-Python: >=3.9.2
 ...
-Classifier: License :: OSI Approved :: Apache Software License
 ...

@cdce8p cdce8p requested a review from a team as a code owner September 23, 2025 12:54
@cdce8p cdce8p requested a review from bmw September 23, 2025 12:54
@bmw bmw self-assigned this Sep 23, 2025
@ohemorange ohemorange requested review from ohemorange and removed request for bmw September 25, 2025 21:50
@ohemorange ohemorange assigned ohemorange and unassigned bmw Sep 25, 2025
@cdce8p cdce8p requested a review from ohemorange October 3, 2025 23:14
Copy link
Contributor

@ohemorange ohemorange left a comment

Choose a reason for hiding this comment

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

Realized we no longer need the comment about keeping up to date with the black section, and added suggestion to reword in keeping with our existing project style.

Co-authored-by: ohemorange <[email protected]>
Copy link
Contributor

@ohemorange ohemorange left a comment

Choose a reason for hiding this comment

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

Looks good to me, thanks for being so responsive here!

@ohemorange ohemorange requested a review from bmw October 3, 2025 23:33
@ohemorange ohemorange assigned bmw and unassigned ohemorange Oct 3, 2025
Copy link
Member

@bmw bmw left a comment

Choose a reason for hiding this comment

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

lgtm as well!

thanks a lot cdce8p

@bmw bmw merged commit e590cb2 into certbot:main Oct 4, 2025
9 checks passed
@cdce8p cdce8p deleted the poetry-v2 branch October 4, 2025 06:07
@cdce8p cdce8p mentioned this pull request Oct 4, 2025
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