Skip to content

Fix import-time coverage - #9284

Merged
crusaderky merged 8 commits into
dask:mainfrom
crusaderky:coverage
Jun 4, 2026
Merged

crusaderky merged 8 commits into
dask:mainfrom
crusaderky:coverage

Conversation

@crusaderky

Copy link
Copy Markdown
Collaborator
  • Fix a CI issue where the coverage report (both local and coveralls) shows as missed everything that is covered by import distributed. This looks like an issue with pytest-cov. I have no idea why the same does not happen in dask/dask too.
  • Fix a coveralls issue where a PR will temporarily be marked as red until the majority of test actions have completed. This aligns dask/distributed to an equivalent hack already in dask/dask.

@crusaderky crusaderky added the tests Unit tests and/or continuous integration label Jun 4, 2026
@github-actions

github-actions Bot commented Jun 4, 2026 •

Copy link
Copy Markdown
Contributor

Unit Test Results

See test report for an extended history of previous test failures. This is useful for diagnosing flaky tests.

    40 files  +    40      40 suites  +40   14h 27m 8s ⏱️ + 14h 27m 8s
 4 150 tests + 4 150   3 971 ✅ + 3 971    179 💤 +  179  0 ❌ ±0 
80 760 runs  +80 760  76 526 ✅ +76 526  4 234 💤 +4 234  0 ❌ ±0 

Results for commit ed6382a. ± Comparison against base commit 3b14fd8.

♻️ This comment has been updated with latest results.

@crusaderky
crusaderky marked this pull request as ready for review June 4, 2026 17:57
@crusaderky
crusaderky requested a review from fjetter as a code owner June 4, 2026 17:57
@crusaderky
crusaderky merged commit 99db80c into dask:main Jun 4, 2026
44 of 47 checks passed
@crusaderky
crusaderky deleted the coverage branch June 4, 2026 17:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tests Unit tests and/or continuous integration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant