Skip to content

Add back 3.7 support#848

Merged
4383 merged 3 commits intomasterfrom
846-reconsider-37-support
Dec 19, 2023
Merged

Add back 3.7 support#848
4383 merged 3 commits intomasterfrom
846-reconsider-37-support

Conversation

@itamarst
Copy link
Copy Markdown
Contributor

Fixes #846

  1. Add back 3.7 support
  2. Make 3.7 tests pass. I'm ... a little worried about this part but I did try to limit it so it mostly just impacts 3.7, except in one case where it was eventlet being incompatible with documented stdlib API.

@itamarst itamarst linked an issue Dec 18, 2023 that may be closed by this pull request
write=True,
timeout=self.gettimeout(),
timeout_exc=timeout_exc('timed out'))
elif _is_py_3_7 and "unexpected eof" in exc.args[1]:
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The error code is 1 in this case, which is Not Helpful.

@codecov
Copy link
Copy Markdown

codecov bot commented Dec 18, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (33e05ca) 53% compared to head (d3098f0) 53%.

Additional details and impacted files
@@          Coverage Diff          @@
##           master   #848   +/-   ##
=====================================
  Coverage      53%    53%           
=====================================
  Files          88     88           
  Lines        9862   9867    +5     
  Branches     1845   1847    +2     
=====================================
+ Hits         5320   5323    +3     
+ Misses       4157   4156    -1     
- Partials      385    388    +3     
Flag Coverage Δ
ipv6 22% <28%> (-1%) ⬇️
py310epolls 52% <28%> (-1%) ⬇️
py310poll 52% <28%> (-1%) ⬇️
py310selects 52% <28%> (-1%) ⬇️
py311epolls 52% <28%> (-1%) ⬇️
py312epolls 50% <28%> (-1%) ⬇️
py37epolls 50% <100%> (?)
py38epolls 52% <28%> (-1%) ⬇️
py38openssl 50% <28%> (-1%) ⬇️
py38poll 52% <28%> (-1%) ⬇️
py38selects 51% <28%> (-1%) ⬇️
py39dnspython1 50% <28%> (-1%) ⬇️
py39epolls 52% <28%> (-1%) ⬇️
py39poll 52% <28%> (-1%) ⬇️
py39selects 51% <28%> (-1%) ⬇️

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.

eventlet_test_ipv6 = 1
deps =
coverage==7.3.3
coverage
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Just use whichever version works with current Python version.

Copy link
Copy Markdown
Member

@4383 4383 left a comment

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Member

@4383 4383 left a comment

Choose a reason for hiding this comment

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

Changelog updated here #849

@4383 4383 merged commit 6441021 into master Dec 19, 2023
@4383 4383 deleted the 846-reconsider-37-support branch December 19, 2023 11:39
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.

Reconsider 3.7 support

3 participants