Skip to content

refactor(cache)!: re-introduce CacheEntry#65

Merged
bassosimone merged 1 commit intomainfrom
fix/gettingdata
Nov 29, 2025
Merged

refactor(cache)!: re-introduce CacheEntry#65
bassosimone merged 1 commit intomainfrom
fix/gettingdata

Conversation

@bassosimone
Copy link
Collaborator

We re-introduce CacheEntry as a container for m-lab data and other companion data. This change makes the library code mostly ready for accommodating other data sources.

BREAKING CHANGE: removed get_mlab_cache_entry and replaced it with a generic get_cache_entry method instead.

We re-introduce CacheEntry as a container for m-lab data and
other companion data. This change makes the library code mostly
ready for accommodating other data sources.

BREAKING CHANGE: removed `get_mlab_cache_entry` and replaced
it with a generic `get_cache_entry` method instead.
@bassosimone bassosimone merged commit 521ccd6 into main Nov 29, 2025
5 checks passed
@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.41%. Comparing base (32d3a6d) to head (404ce30).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #65   +/-   ##
=======================================
  Coverage   82.41%   82.41%           
=======================================
  Files          27       27           
  Lines        2257     2257           
=======================================
  Hits         1860     1860           
  Misses        397      397           

☔ View full report in Codecov by Sentry.
📢 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.

@bassosimone bassosimone deleted the fix/gettingdata branch November 29, 2025 13:36
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