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 assertion in GHA tests #670

Merged
merged 2 commits into from
Nov 24, 2022
Merged

Fix assertion in GHA tests #670

merged 2 commits into from
Nov 24, 2022

Conversation

FollowTheProcess
Copy link
Collaborator

I noticed that on the recent release, the action_default_tests were failing asserting that the calling python was 3.10 when in fact it was 3.11 due to #667

This PR updates the assertion to 3.11

@FollowTheProcess FollowTheProcess added the ci Issues relating to nox's CI pipeline label Nov 21, 2022
@FollowTheProcess FollowTheProcess self-assigned this Nov 21, 2022
@FollowTheProcess
Copy link
Collaborator Author

FollowTheProcess commented Nov 21, 2022

Hmm looks like Ubuntu 18.04 still defaults to 3.10?

Edit: Looking at the docs it would appear Ubuntu-18.04 is deprecated so I've removed it in this PR

@FollowTheProcess
Copy link
Collaborator Author

FollowTheProcess commented Nov 21, 2022

Also, only just noticed but we only run actions_all_tests on Windows for some reason? If we only want 1 OS, would ubuntu-latest not make more sense?

runs-on: windows-latest

@FollowTheProcess
Copy link
Collaborator Author

Unless anyone objects I'm probably just going to merge this as it keeps breaking workflows on main which I'm not a fan of 👍🏻

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Issues relating to nox's CI pipeline
Development

Successfully merging this pull request may close these issues.

1 participant