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: Drop 3.8 and 3.9 workflows #1822

Closed
wants to merge 4 commits into from

Conversation

alwaysintreble
Copy link
Collaborator

What is this fixing or adding?

Drops 3.8 and 3.9 support and adds 3.11 support in unit tests. We need per-game datapackage and #1821 before this gets merged but there shouldn't be any conflicts.

How was this tested?

The power of tests

@ThePhar ThePhar added is: refactor/cleanup Improvements to code/output readability or organizization. affects: core Issues/PRs that touch core and may need additional validation. labels May 31, 2023
@ThePhar ThePhar added is: maintenance Regular updates to requirements and utilities that do not fix bugs or change/add features. and removed is: refactor/cleanup Improvements to code/output readability or organizization. affects: core Issues/PRs that touch core and may need additional validation. labels Jul 4, 2023
@ThePhar
Copy link
Member

ThePhar commented Jul 6, 2023

This should be broken up into 2 different stages. Add 3.11 workflow first since next AP release is 3.11 so we should be checking for that now. Then after we drop win 7 support, remove 3.8 and 3.9 workflows.

I've done the first part in #1949 and outlined plan there. Going to leave this PR open for removing 3.8 and 3.9 workflows once we get to that stage.

@ThePhar ThePhar changed the title Tests: Drop 3.8 and 3.9; add 3.11 tests CI: Drop 3.8 and 3.9; add 3.11 workflow Jul 6, 2023
# Conflicts:
#	.github/workflows/unittests.yml
@alwaysintreble alwaysintreble changed the title CI: Drop 3.8 and 3.9; add 3.11 workflow CI: Drop 3.8 and 3.9 workflows Jul 24, 2023
@alwaysintreble
Copy link
Collaborator Author

Not sure if we want 3.10 windows and macos workflows, but as there aren't any currently didn't add any, and don't think we have much reason to since all distributed builds will be on 3.11 anyways?

@PoryGone PoryGone added the waiting-on: other Issue/PR is waiting for something else, like another PR. label Mar 6, 2024
@BadMagic100 BadMagic100 added waiting-on: peer-review Issue/PR has not been reviewed by enough people yet. and removed waiting-on: other Issue/PR is waiting for something else, like another PR. labels Aug 20, 2024
@BadMagic100
Copy link
Collaborator

I think having builds for all supported versions across all OSs would be ideal until/unless the available CI hours for the org run out

@alwaysintreble alwaysintreble added this to the Archipelago 0.6.0 milestone Nov 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
is: maintenance Regular updates to requirements and utilities that do not fix bugs or change/add features. waiting-on: peer-review Issue/PR has not been reviewed by enough people yet.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants