Skip to content

Drop support for Python 3.8 #1203

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

Merged
merged 14 commits into from
Oct 13, 2024
Merged

Drop support for Python 3.8 #1203

merged 14 commits into from
Oct 13, 2024

Conversation

bdraco
Copy link
Member

@bdraco bdraco commented Oct 11, 2024

DNM: Should do a patch release to fix the performance regression in #1198 before dropping 3.8

What do these changes do?

Python 3.8 has reached end of life
https://devguide.python.org/versions/

Are there changes in behavior for the user?

Remove support for Python 3.8

@psf-chronographer psf-chronographer bot added the bot:chronographer:provided There is a change note present in this PR label Oct 11, 2024
@bdraco bdraco changed the title Drop support for Python 3.8 DNM: Drop support for Python 3.8 Oct 11, 2024
Copy link

codecov bot commented Oct 11, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.71%. Comparing base (8243b02) to head (474e530).
Report is 181 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1203      +/-   ##
==========================================
- Coverage   95.85%   95.71%   -0.14%     
==========================================
  Files          27       27              
  Lines        5182     5181       -1     
  Branches      317      298      -19     
==========================================
- Hits         4967     4959       -8     
- Misses        189      195       +6     
- Partials       26       27       +1     
Flag Coverage Δ
CI-GHA 95.71% <100.00%> (-0.12%) ⬇️
MyPy 43.92% <94.73%> (-0.02%) ⬇️
OS-Linux 99.28% <100.00%> (-0.08%) ⬇️
OS-Windows 99.34% <100.00%> (-0.08%) ⬇️
OS-macOS 98.99% <100.00%> (-0.08%) ⬇️
Py-3.10.11 98.97% <100.00%> (-0.01%) ⬇️
Py-3.10.15 99.23% <100.00%> (-0.01%) ⬇️
Py-3.11.10 99.23% <100.00%> (-0.01%) ⬇️
Py-3.11.9 98.97% <100.00%> (-0.01%) ⬇️
Py-3.12.7 99.23% <100.00%> (-0.01%) ⬇️
Py-3.13.0 99.23% <100.00%> (-0.01%) ⬇️
Py-3.8.10 ?
Py-3.8.18 ?
Py-3.9.13 98.92% <100.00%> (-0.01%) ⬇️
Py-3.9.20 99.18% <100.00%> (-0.01%) ⬇️
Py-pypy7.3.11 ?
Py-pypy7.3.16 99.24% <100.00%> (-0.01%) ⬇️
Py-pypy7.3.17 99.26% <100.00%> (-0.01%) ⬇️
VM-macos-latest 98.99% <100.00%> (-0.08%) ⬇️
VM-ubuntu-latest 99.28% <100.00%> (-0.08%) ⬇️
VM-windows-latest 99.34% <100.00%> (-0.08%) ⬇️
pytest 99.28% <100.00%> (-0.08%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@bdraco bdraco force-pushed the drop_python3.8 branch 2 times, most recently from 744c9af to 91b9274 Compare October 12, 2024 00:09
@bdraco bdraco mentioned this pull request Oct 12, 2024
@bdraco bdraco changed the title DNM: Drop support for Python 3.8 Drop support for Python 3.8 Oct 12, 2024
@bdraco bdraco marked this pull request as ready for review October 12, 2024 02:55
Copy link

codspeed-hq bot commented Oct 12, 2024

CodSpeed Performance Report

Merging #1203 will not alter performance

Comparing drop_python3.8 (474e530) with master (33294bf)

Summary

✅ 45 untouched benchmarks

@bdraco bdraco closed this Oct 12, 2024
@bdraco bdraco reopened this Oct 12, 2024
@bdraco bdraco closed this Oct 12, 2024
@bdraco bdraco reopened this Oct 12, 2024
@bdraco bdraco marked this pull request as draft October 12, 2024 05:18
@bdraco
Copy link
Member Author

bdraco commented Oct 12, 2024

I'll leave this for a few days in case we need to publish a patch to fix any regressions.

@bdraco bdraco closed this Oct 12, 2024
@bdraco bdraco reopened this Oct 12, 2024
@bdraco bdraco force-pushed the drop_python3.8 branch 2 times, most recently from 443a4de to 050dda7 Compare October 12, 2024 16:31
@bdraco bdraco reopened this Oct 12, 2024
@bdraco
Copy link
Member Author

bdraco commented Oct 12, 2024

master vs 1.15.0 shows ~17% improvement in HA. Probably worth doing one more patch before merging this so py3.8 gets the changes as well

@bdraco bdraco closed this Oct 13, 2024
@bdraco bdraco reopened this Oct 13, 2024
@bdraco
Copy link
Member Author

bdraco commented Oct 13, 2024

All the rough edges identified by adding the benchmarks have been smoothed out. 1.15.2 looks like it will be the last release to support Python 3.8

@bdraco bdraco marked this pull request as ready for review October 13, 2024 18:49
@bdraco
Copy link
Member Author

bdraco commented Oct 13, 2024

Last gate is to wait for home-assistant/core#128309 to pass

@bdraco bdraco merged commit b64f231 into master Oct 13, 2024
43 of 46 checks passed
@bdraco bdraco deleted the drop_python3.8 branch October 13, 2024 19:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bot:chronographer:provided There is a change note present in this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant