Skip to content

Conversation

fu220
Copy link
Contributor

@fu220 fu220 commented Aug 6, 2025

Description

This pull request describes the modification of comments on Cache Storage, clarifying the specific use cases of Cache Storage.

Related Issue

Motivation and Context

Screenshots (if appropriate)

Modify Cache storage comments to clarify its usage scenarios.

Signed-off-by: fu220 <2863318196@qq.com>
Copy link

codecov bot commented Aug 6, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 48.43%. Comparing base (e1ae65a) to head (717fd56).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1277      +/-   ##
==========================================
- Coverage   48.47%   48.43%   -0.05%     
==========================================
  Files          60       60              
  Lines       15011    15011              
==========================================
- Hits         7277     7270       -7     
- Misses       7734     7741       +7     
Files with missing lines Coverage Δ
dragonfly-client-config/src/dfdaemon.rs 92.22% <ø> (ø)
dragonfly-client-storage/src/cache/mod.rs 99.51% <ø> (ø)

... and 1 file with indirect coverage changes

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

Copy link
Member

@gaius-qi gaius-qi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@gaius-qi gaius-qi added the enhancement New feature or request label Aug 6, 2025
Copy link
Contributor

@LunaWhispers LunaWhispers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@LunaWhispers LunaWhispers merged commit 1f8a323 into dragonflyoss:main Aug 6, 2025
8 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants