search

JuliaGraphs / Graphs.jl

An optimized graphs package for the Julia programming language
http://juliagraphs.org/Graphs.jl/
Other
451 stars 87 forks source link

fix merge_vertices #369

Closed etiennedeg closed 2 months ago

etiennedeg commented 3 months ago

Fixes #363

In the tests, there was a loop encapsulating the merge_vertices tests, but these tests were not using the test graph, hence I moved it out and refactored it a bit.

codecov[bot] commented 2 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 97.27%. Comparing base (afb8245) to head (cdb589b). Report is 2 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #369 +/- ## ========================================== - Coverage 97.28% 97.27% -0.01% ========================================== Files 118 118 Lines 6876 6874 -2 ========================================== - Hits 6689 6687 -2 Misses 187 187 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

gdalle commented 2 months ago

feel free to merge it @etiennedeg, according to ColPrac the person submitting the PR should be the one to merge it after review if they have the rights