Closed eliarbel closed 3 months ago
@mtreinish this is ready for review now, however CI clippy
complains on parts in the code not included in this PR (with error: usage of a legacy numeric constant
). Any idea?
My guess is that the release of rust 1.79 last week is causing new clippy rules to be run and that's causing the failure. Although I thought we should have everything pinned to the MSRV in CI to avoid this kind of situation.
My guess is that the release of rust 1.79 last week is causing new clippy rules to be run and that's causing the failure. Although I thought we should have everything pinned to the MSRV in CI to avoid this kind of situation.
Yes, I was on 1.77. Updating to 1.79 I see those warnings locally as well. I'll wait for it to be resolved.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
---|---|---|---|---|---|
rustworkx-core/src/dag_algo.rs | 68 | 70 | 97.14% | ||
<!-- | Total: | 82 | 84 | 97.62% | --> |
Files with Coverage Reduction | New Missed Lines | % | ||
---|---|---|---|---|
src/connectivity/mod.rs | 16 | 96.75% | ||
<!-- | Total: | 16 | --> |
Totals | |
---|---|
Change from base Build 9565715860: | 0.03% |
Covered Lines: | 17460 |
Relevant Lines: | 18214 |
Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
---|---|---|---|---|---|
rustworkx-core/src/dag_algo.rs | 68 | 70 | 97.14% | ||
<!-- | Total: | 82 | 84 | 97.62% | --> |
Totals | |
---|---|
Change from base Build 9589302565: | 0.0% |
Covered Lines: | 17460 |
Relevant Lines: | 18214 |
This PR moves
collect_runs()
functionality to rustworkx-core intorustworkx-core/src/dag_algo.rs
. This will be used bycollect_runs()
insrc/dag_algo/mod.rs
.Closes #1199
Tasks: