Skip to content

Split out dask/dask tests in CI#808

Merged
crusaderky merged 3 commits into
dask:mainfrom
crusaderky:split_dask_tests
Jan 25, 2024
Merged

Split out dask/dask tests in CI#808
crusaderky merged 3 commits into
dask:mainfrom
crusaderky:split_dask_tests

Conversation

@crusaderky

@crusaderky crusaderky commented Jan 25, 2024

Copy link
Copy Markdown
Collaborator

The dask/dask tests never got to run if there's a failure in the dask-expr tests.
This PR also reduces end-to-end runtime by 3 minutes.

@crusaderky crusaderky changed the title Split out dask/daskl tests in CI Split out dask/dask tests in CI Jan 25, 2024

@hendrikmakait hendrikmakait left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks, @crusaderky!

Comment thread .github/workflows/dask_test.yaml Outdated
Co-authored-by: Hendrik Makait <hendrik@makait.com>
@crusaderky crusaderky merged commit e32ab95 into dask:main Jan 25, 2024
@crusaderky crusaderky deleted the split_dask_tests branch January 25, 2024 18:12
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.

2 participants