Skip to content

Removes unused param from Bank::run_final_hash_calc()#6989

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

Removes unused param from Bank::run_final_hash_calc()#6989
brooksprumo merged 1 commit intoanza-xyz:masterfrom
brooksprumo:lthash/215/run_final_hash_calc

Conversation

@brooksprumo
Copy link
Copy Markdown

Problem

The on_halt_store_hash_raw_data_for_debug param for Bank::run_final_hash_calc() is unused and should be removed.

Summary of Changes

Remove it and update callers.

@brooksprumo brooksprumo self-assigned this Jul 16, 2025
@brooksprumo brooksprumo marked this pull request as ready for review July 16, 2025 02:49
@brooksprumo brooksprumo requested review from HaoranYi and roryharr July 16, 2025 02:49
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jul 16, 2025

Codecov Report

❌ Patch coverage is 0% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 83.2%. Comparing base (78d5954) to head (1853bcb).
⚠️ Report is 2910 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #6989   +/-   ##
=======================================
  Coverage    83.2%    83.2%           
=======================================
  Files         853      853           
  Lines      375531   375524    -7     
=======================================
- Hits       312586   312582    -4     
+ Misses      62945    62942    -3     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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 1fe5ac4 into anza-xyz:master Jul 16, 2025
41 checks passed
@brooksprumo brooksprumo deleted the lthash/215/run_final_hash_calc branch July 16, 2025 16:31
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