@echan5 Thanks for the PR. What tests were failing and what is your environment? Tests are not failing on our CI so it must be your local env setup.

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.08%. Comparing base (0978fb8) to head (9a673fe).
⚠️ Report is 8 commits behind head on dev.

@@           Coverage Diff           @@
##              dev     #571   +/-   ##
=======================================
  Coverage   96.07%   96.08%           
=======================================
  Files          16       16           
  Lines        4463     4465    +2     
=======================================
+ Hits         4288     4290    +2     
  Misses        175      175           

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

You're right - it was my setup. It didn't originally occur to me how the dependencies of this repo are structured and I had just run uv sync --extra test (using python 3.12) before running uv run pytest tests (and adding any missing imports that came up like uv pip install numpy pytz click uuid6 pydantic and had gotten various assertion errors like the below (the list was actually much longer), which at first glance seemed to be discrepancies with spacing in strings:

FAILED tests/test_delta.py::TestBasicsOfDelta::test_delta_repr - assert '<Delta: {"iterable_item_added": {"root[2]": 3, "root[3]": 5}}>' in {'<Delta: {"iterable_item_added":{"root[2]":3,"root[3]":5}}>...

FAILED tests/test_summarize.py::TestSummarize::test_list_summary - assert '[100,101,102,103,10,"..."]' == '[100, 101, 1...3, 10, "..."]'

But I just examined more closely and realized I should re-install the environment with uv sync --all-extras and all the tests pass now. I'll also revert the changes I had put into pyproject.toml/uv.lock thinking they were to solve these dependency issues.

Please let me know if anything else needs addressing - thank you!

Hi @echan5 Thanks for the PR. Reviewing shortly.