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

ci: only use Python 3.10+ #827

Merged
merged 3 commits into from
Aug 26, 2024
Merged

ci: only use Python 3.10+ #827

merged 3 commits into from
Aug 26, 2024

Conversation

lengau
Copy link
Contributor

@lengau lengau commented Aug 22, 2024

This was already happening thanks to tox changes, but this does do unit tests on 3.13 on focal now too.

  • Have you signed the CLA?

This was already happening thanks to tox changes, but this does
do unit tests on 3.13 on focal now too.
@lengau lengau requested review from tigarmo and a team August 22, 2024 18:05
@tigarmo tigarmo requested a review from a team August 22, 2024 18:05
@tigarmo
Copy link
Contributor

tigarmo commented Aug 22, 2024

Please fix the "unly" on the commit message (😆)

@lengau lengau changed the title ci: unly use Python 3.10+ ci: only use Python 3.10+ Aug 22, 2024
@lengau lengau requested a review from tigarmo August 22, 2024 18:08
@lengau
Copy link
Contributor Author

lengau commented Aug 22, 2024

@tigarmo fixed. Also re-requested you since I did some other cleanup too

@tigarmo tigarmo requested a review from a team August 22, 2024 18:12
@cmatsuoka
Copy link
Collaborator

I think we'll need @mr-cal's assistance to fix the required tests list in order to land this.

@lengau
Copy link
Contributor Author

lengau commented Aug 23, 2024

@cmatsuoka could you change the required ci items to match my typo fix?

@cmatsuoka cmatsuoka merged commit 5b90250 into main Aug 26, 2024
10 checks passed
@cmatsuoka cmatsuoka deleted the ci/remove-focal branch August 26, 2024 15:55
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