Skip to content

ledger-tool: Allow compute-slot-cost to operate on dead slots#213

Merged
steviez merged 1 commit intoanza-xyz:masterfrom
steviez:lt_cc_allow_dead
Mar 13, 2024
Merged

ledger-tool: Allow compute-slot-cost to operate on dead slots#213
steviez merged 1 commit intoanza-xyz:masterfrom
steviez:lt_cc_allow_dead

Conversation

@steviez
Copy link
Copy Markdown

@steviez steviez commented Mar 12, 2024

Summary of Changes

Make this command accept the --allow-dead-slots arg as well

Make this command accept the --allow-dead-slots arg as well
@steviez steviez requested a review from tao-stones March 12, 2024 22:00
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.9%. Comparing base (2ddb50d) to head (abbd5da).
Report is 9 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##           master     #213     +/-   ##
=========================================
- Coverage    81.9%    81.9%   -0.1%     
=========================================
  Files         837      837             
  Lines      226518   226518             
=========================================
- Hits       185551   185536     -15     
- Misses      40967    40982     +15     

Copy link
Copy Markdown

@tao-stones tao-stones left a comment

Choose a reason for hiding this comment

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

lgtm.

@steviez steviez merged commit 5ed30be into anza-xyz:master Mar 13, 2024
@steviez steviez deleted the lt_cc_allow_dead branch March 13, 2024 16:29
codebender828 pushed a commit to codebender828/agave that referenced this pull request Oct 3, 2024
…yz#213)

Make this command accept the --allow-dead-slots arg as well
OliverNChalk pushed a commit to OliverNChalk/agave that referenced this pull request Nov 11, 2025
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