Skip to content

Handle self-loops in all_simple_paths (backport #1619)#1631

Merged
IvanIsCoding merged 2 commits into
stable/0.18from
mergify/bp/stable/0.18/pr-1619
Jul 10, 2026
Merged

Handle self-loops in all_simple_paths (backport #1619)#1631
IvanIsCoding merged 2 commits into
stable/0.18from
mergify/bp/stable/0.18/pr-1619

Conversation

@mergify

@mergify mergify Bot commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Closes #1617

This was a discrepancy introduced in #1520. For the record, NetworkX returns ["a"] for a self-loop which is slightly different than ["a", "a"] but I think the old behaviour is fine.


This is an automatic backport of pull request #1619 done by Mergify.

@coveralls

coveralls commented Jul 9, 2026

Copy link
Copy Markdown

Coverage Report for CI Build 29058793256

Coverage decreased (-0.02%) to 94.696%

Details

  • Coverage decreased (-0.02%) from the base build.
  • Patch coverage: 4 uncovered changes across 1 file (18 of 22 lines covered, 81.82%).
  • No coverage regressions found.

Uncovered Changes

File Changed Covered %
src/connectivity/mod.rs 22 18 81.82%

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 20307
Covered Lines: 19230
Line Coverage: 94.7%
Coverage Strength: 920401.97 hits per line

💛 - Coveralls

@mergify

mergify Bot commented Jul 10, 2026

Copy link
Copy Markdown
Contributor Author

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

  • Queue this pull request

@IvanIsCoding IvanIsCoding merged commit af888af into stable/0.18 Jul 10, 2026
40 checks passed
@IvanIsCoding IvanIsCoding deleted the mergify/bp/stable/0.18/pr-1619 branch July 10, 2026 00:37
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