Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

ci: remove skipped job & combine (faster) #771

Merged
merged 2 commits into from
Feb 23, 2024

Conversation

henryiii
Copy link
Collaborator

@henryiii henryiii commented Feb 20, 2024

Pulling out of #762. Updates CI. Mostly a followup to #673.

Also avoiding running the entire test suite twice, just the tox-to-nox part.

@henryiii henryiii changed the title ci: fix coverage ci: remove skipped job Feb 20, 2024
@henryiii henryiii marked this pull request as ready for review February 20, 2024 21:25
@henryiii henryiii changed the title ci: remove skipped job ci: remove skipped job & combine (faster) Feb 22, 2024
@henryiii
Copy link
Collaborator Author

I think rebasing on #778 will be good. This saves at least a minute per previously pair of runs.

Signed-off-by: Henry Schreiner <henryschreineriii@gmail.com>
Signed-off-by: Henry Schreiner <henryschreineriii@gmail.com>
@henryiii henryiii merged commit 78a2612 into wntrblm:main Feb 23, 2024
22 checks passed
@henryiii henryiii deleted the henryiii/ci/fixcov branch February 23, 2024 15:58
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants