Skip to content

Comments

chore: Simplify storage_from_config#2597

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

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

Conversation

@Felixoid
Copy link
Contributor

@Felixoid Felixoid commented Feb 8, 2026

It's another part of preparation for #2581

The diff is tiny, when Hide whitespace is used

@Felixoid
Copy link
Contributor Author

Felixoid commented Feb 8, 2026

image

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

codecov-commenter commented Feb 8, 2026

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