Skip to content

status_cache: add comments and move away test only code#8289

Merged
alessandrod merged 1 commit intoanza-xyz:masterfrom
alessandrod:status-chore
Oct 2, 2025
Merged

status_cache: add comments and move away test only code#8289
alessandrod merged 1 commit intoanza-xyz:masterfrom
alessandrod:status-chore

Conversation

@alessandrod
Copy link
Copy Markdown

another chunk of #3796

Adds some comments, moves Eq impl to mod tests and marks clear as dcou.

Copy link
Copy Markdown

@bw-solana bw-solana left a comment

Choose a reason for hiding this comment

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

LGTM

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 80.00000% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 83.2%. Comparing base (21d88de) to head (10f30f5).
⚠️ Report is 4 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #8289   +/-   ##
=======================================
  Coverage    83.2%    83.2%           
=======================================
  Files         836      836           
  Lines      366683   366682    -1     
=======================================
+ Hits       305278   305292   +14     
+ Misses      61405    61390   -15     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@alessandrod alessandrod merged commit 0ee8338 into anza-xyz:master Oct 2, 2025
43 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