Skip to content

chore: Simplify storage_from_config#2597

Merged
sylvestre merged 1 commit into
mozilla:mainfrom
Felixoid:simplify-storage_from_config
Feb 9, 2026
Merged

chore: Simplify storage_from_config#2597
sylvestre merged 1 commit into
mozilla:mainfrom
Felixoid:simplify-storage_from_config

Conversation

@Felixoid

@Felixoid Felixoid commented Feb 8, 2026

Copy link
Copy Markdown
Contributor

It's another part of preparation for #2581

The diff is tiny, when Hide whitespace is used

@Felixoid

Felixoid commented Feb 8, 2026

Copy link
Copy Markdown
Contributor Author
image

@Felixoid
Felixoid force-pushed the simplify-storage_from_config branch from 187eec0 to 604a674 Compare February 8, 2026 21:36
@codecov-commenter

codecov-commenter commented Feb 8, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 35.03185% with 102 lines in your changes missing coverage. Please review.
✅ Project coverage is 72.17%. Comparing base (189ac78) to head (604a674).

Files with missing lines Patch % Lines
src/cache/cache.rs 35.03% 102 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2597      +/-   ##
==========================================
+ Coverage   72.06%   72.17%   +0.11%     
==========================================
  Files          68       68              
  Lines       37011    36985      -26     
==========================================
+ Hits        26672    26694      +22     
+ Misses      10339    10291      -48     

☔ 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.

@sylvestre

Copy link
Copy Markdown
Collaborator

The diff is tiny, when Hide whitespace is used

TIL, thanks!

@sylvestre
sylvestre merged commit 0e43057 into mozilla:main Feb 9, 2026
49 checks passed
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.

3 participants