Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Introduce Myers algorithm in linear space #112

Merged
merged 14 commits into from
Jun 11, 2024

refactor: remove unnecessary list copying

37933be
Select commit
Loading
Failed to load commit list.
Merged

Introduce Myers algorithm in linear space #112

refactor: remove unnecessary list copying
37933be
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Jun 5, 2024 in 0s

86.07% (+1.80%) compared to 875f4ce

View this Pull Request on Codecov

86.07% (+1.80%) compared to 875f4ce

Details

Codecov Report

Attention: Patch coverage is 94.21769% with 17 lines in your changes missing coverage. Please review.

Project coverage is 86.07%. Comparing base (875f4ce) to head (37933be).

Files Patch % Lines
...github/petertrr/diffutils/text/DiffRowGenerator.kt 93.61% 3 Missing and 6 partials ⚠️
...futils/algorithm/myers/MyersDiffWithLinearSpace.kt 92.78% 1 Missing and 6 partials ⚠️
...hub/petertrr/diffutils/algorithm/myers/PathNode.kt 80.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files 
@@             Coverage Diff              @@
##               main     #112      +/-   ##
============================================
+ Coverage     84.26%   86.07%   +1.80%     
- Complexity      151      221      +70     
============================================
  Files            22       30       +8     
  Lines           534      632      +98     
  Branches         89      117      +28     
============================================
+ Hits            450      544      +94     
- Misses           57       59       +2     
- Partials         27       29       +2

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

View more details on Codecov