Skip to content

Removes unused on_halt_store_hash_raw_data_for_debug#6990

Merged
brooksprumo merged 1 commit intoanza-xyz:masterfrom
brooksprumo:lthash/215/on_halt_store_hash_raw_data_for_debug
Jul 16, 2025
Merged

Removes unused on_halt_store_hash_raw_data_for_debug#6990
brooksprumo merged 1 commit intoanza-xyz:masterfrom
brooksprumo:lthash/215/on_halt_store_hash_raw_data_for_debug

Conversation

@brooksprumo
Copy link
Copy Markdown

@brooksprumo brooksprumo commented Jul 16, 2025

Problem

The on_halt_store_hash_raw_data_for_debug ledger-tool cli arg is unused and should be removed.

Summary of Changes

Removes it and fixes up callers.

@brooksprumo brooksprumo self-assigned this Jul 16, 2025
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jul 16, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.2%. Comparing base (1fe5ac4) to head (9403961).
Report is 3 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##           master    #6990     +/-   ##
=========================================
- Coverage    83.2%    83.2%   -0.1%     
=========================================
  Files         853      853             
  Lines      375401   375392      -9     
=========================================
- Hits       312545   312487     -58     
- Misses      62856    62905     +49     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@brooksprumo brooksprumo force-pushed the lthash/215/on_halt_store_hash_raw_data_for_debug branch from 2f080c8 to 9403961 Compare July 16, 2025 16:46
Comment thread ledger/src/blockstore_processor.rs
@brooksprumo brooksprumo marked this pull request as ready for review July 16, 2025 17:18
@brooksprumo brooksprumo requested review from HaoranYi and roryharr July 16, 2025 17:18
Copy link
Copy Markdown

@HaoranYi HaoranYi left a comment

Choose a reason for hiding this comment

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

lgtm.

@brooksprumo brooksprumo merged commit 4cb7c4a into anza-xyz:master Jul 16, 2025
41 checks passed
@brooksprumo brooksprumo deleted the lthash/215/on_halt_store_hash_raw_data_for_debug branch July 16, 2025 19:44
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.

4 participants