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

Drop support for Python 3.8 #28

Merged
merged 1 commit into from
Jan 8, 2025
Merged

Drop support for Python 3.8 #28

merged 1 commit into from
Jan 8, 2025

Conversation

johannaengland
Copy link
Contributor

@johannaengland johannaengland commented Jan 7, 2025

To make working on #20 easier.

Argus itself does not support 3.8 anymore
Copy link

sonarqubecloud bot commented Jan 7, 2025

Copy link

github-actions bot commented Jan 7, 2025

Test results

 5 files   5 suites   2s ⏱️
 2 tests  2 ✅ 0 💤 0 ❌
10 runs  10 ✅ 0 💤 0 ❌

Results for commit 23b2524.

@codecov-commenter
Copy link

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

Copy link
Member

@lunkwill42 lunkwill42 left a comment

Choose a reason for hiding this comment

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

Argus itself does not support 3.8 anymore.

IMO, this is by itself not a valid reason to drop 3.8 support in pyargus. Which versions of Python Argus runs on is irrelevant to pyargus.

However, once it becomes an uphill battle to keep support for 3.8 in pyargus, we can drop it. I believe you used the same reasoning for dropping 3.7 in #12, but that was also premature (but then, GitHub Actions ubuntu-latest image made testing on 3.7 difficult just a few months later, and it was time)

Copy link
Member

@lunkwill42 lunkwill42 left a comment

Choose a reason for hiding this comment

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

To make working on #20 easier.

Well, see, a reference to jog my post-xmas-memory. Much better 😆

@johannaengland johannaengland merged commit b9457b2 into master Jan 8, 2025
11 checks passed
@johannaengland johannaengland deleted the stop-tests-py38 branch January 8, 2025 08:23
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