Skip to content

Conversation

@smarnach
Copy link
Contributor

Version 8.2.0 of click introduces incompatible changes to the CliRunner test class while at the same time increasing the minimum supported Python version to 3.10. crahsstats-tools still supports Python 3.9, so we'll use click earlier than 8.2.0 for Python 3.9 and newer versions of click for newer versions of Python. Since the changes to CliRunner are backwards-incompatible, we'd need different versions of the test code for different versions of Python. To simplify things, I'll just pin click to the old version for now, and we can properly fix this once we drop support for Python 3.9 later this year.

Version 8.2.0 of click introduces incompatible changes to the CliRunner
test class while at the same time increasing the minimum supported
Python version to 3.10. crahsstats-tools still supports Python 3.9, so
we'll use click earlier than 8.2.0 for Python 3.9 and newer versions of
click for newer versions of Python. Since the changes to CliRunner are
backwards-incompatible, we'd need different versions of the test code
for different versions of Python. To simplify things, I'll just pin
click to the old version for now, and we can properly fix this once we
drop support for Python 3.9 later this year.
@smarnach smarnach requested a review from a team May 26, 2025 18:16
Copy link

@leplatrem leplatrem left a comment

Choose a reason for hiding this comment

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

For future reference, Python 3.9 supports ends in 5 months: https://endoflife.date/python (31/10/25)

@smarnach smarnach merged commit 914856b into main Jun 3, 2025
5 checks passed
@smarnach smarnach deleted the pin-click branch June 3, 2025 14:12
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