Skip to content

feat: align Sequences with difflib before being diffed#17

Open
dpinol wants to merge 1 commit into
Solganis:mainfrom
dpinol:align-diffs
Open

feat: align Sequences with difflib before being diffed#17
dpinol wants to merge 1 commit into
Solganis:mainfrom
dpinol:align-diffs

Conversation

@dpinol

@dpinol dpinol commented Jul 25, 2026

Copy link
Copy Markdown
Contributor

Now [0, 1, 2, 3] vs [1, 2, 3] produces exactly one DiffEntry — [0]: - 0.

Now [0, 1, 2, 3] vs [1, 2, 3] produces exactly one DiffEntry — [0]: - 0.
@codspeed-hq

codspeed-hq Bot commented Jul 25, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 9 untouched benchmarks


Comparing dpinol:align-diffs (f0c228c) with main (2349734)

Open in CodSpeed

@codecov

codecov Bot commented Jul 25, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (2349734) to head (f0c228c).

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #17   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           34        34           
  Lines         4713      4755   +42     
  Branches      1020      1030   +10     
=========================================
+ Hits          4713      4755   +42     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Solganis

Copy link
Copy Markdown
Owner

Sorry, can't merge it without running this code myself. I will review it asap after the vacation.

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