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

Fix Python3.6 unittest by using ubuntu 20.04 #272

Merged
merged 4 commits into from
Dec 9, 2022

Conversation

giffels
Copy link
Member

@giffels giffels commented Dec 7, 2022

According to actions/runner-images#6399 GitHub's run-on ubuntu-latest tag is now pointing to Ubuntu 22.04. However, this version does not offer a Python 3.6 interpreter according to actions/setup-python#544. This will break our unittest for the good old Python 3.6 version.

This pull request forces GitHub actions to use Ubuntu 20.04 instead, which still supports Python 3.6. Just another hint that it is time to get rid of it. 😌

In addition, the newest flake checks for PEP-618, which is only supported for Python>=3.10. Hence, I have disabled the checks on the concerned lines.

@codecov-commenter
Copy link

codecov-commenter commented Dec 7, 2022

Codecov Report

Base: 98.88% // Head: 98.88% // No change to project coverage 👍

Coverage data is based on head (d5fb585) compared to base (7ddc223).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #272   +/-   ##
=======================================
  Coverage   98.88%   98.88%           
=======================================
  Files          56       56           
  Lines        2325     2325           
=======================================
  Hits         2299     2299           
  Misses         26       26           
Impacted Files Coverage Δ
tardis/utilities/asyncbulkcall.py 96.15% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@giffels giffels marked this pull request as ready for review December 7, 2022 11:54
@giffels giffels requested review from a team, eileen-kuehn and rfvc and removed request for a team December 7, 2022 11:55
Copy link
Member

@maxfischer2781 maxfischer2781 left a comment

Choose a reason for hiding this comment

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

Simple enough!

I've added a ticket to start tracking workarounds specifically for Python 3.6.

@rfvc
Copy link
Contributor

rfvc commented Dec 8, 2022

Great! Just for reference, the versions of Python available in the Ubuntu 20.04-runner and for Ubuntu 22.04

@giffels giffels merged commit 9c6f391 into MatterMiners:master Dec 9, 2022
@giffels giffels deleted the fix-python36-unittests branch December 9, 2022 09:46
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.

4 participants