Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@O-sama12
Copy link

Fixed the issue #11838 where user can land the exact documentation page for dask.

@github-actions
Copy link
Contributor

github-actions bot commented Apr 14, 2025

Unit Test Results

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

      9 files  ±0        9 suites  ±0   3h 17m 26s ⏱️ +47s
 18 012 tests ±0   16 798 ✅ ±0   1 214 💤 ±0  0 ❌ ±0 
161 182 runs  ±0  149 074 ✅  - 4  12 108 💤 +4  0 ❌ ±0 

Results for commit a99c862. ± Comparison against base commit 8639b6e.

♻️ This comment has been updated with latest results.

@a-holm
Copy link

a-holm commented Apr 14, 2025

LGTM, looks like it solves the issue

@O-sama12
Copy link
Author

Yeah, it literally fix the issue.

@O-sama12
Copy link
Author

Hey @a-holm
It's day 5 since you've messaged me, and I am commenting you again showing my concerns whether it really solves the issue or not since you haven't merged my pull request?

Please le me know when you are merging my pull request with the main codebase?

With regards
@O-sama12

@a-holm
Copy link

a-holm commented Apr 19, 2025

@O-sama12 There are many pull requests that solves the issue, i also have a pull request that solves the issue. If it was up to me I would have merged my own pull request long ago before you made yours. But it solves the issue, yes.

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