Skip to content

feat: Temporary cap bitcoin-da fee rate#2942

Open
jfldde wants to merge 10 commits intonightlyfrom
feat/cap-da-max-fee-rate
Open

feat: Temporary cap bitcoin-da fee rate#2942
jfldde wants to merge 10 commits intonightlyfrom
feat/cap-da-max-fee-rate

Conversation

@jfldde
Copy link
Contributor

@jfldde jfldde commented Sep 30, 2025

Description

  • Cap bitcoin-da fee rate via config. Defaults to 15 sat/vb

Linked Issues

@codecov
Copy link

codecov bot commented Sep 30, 2025

Codecov Report

❌ Patch coverage is 68.42105% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 85.8%. Comparing base (4342724) to head (23a4ee8).
⚠️ Report is 3 commits behind head on nightly.

Files with missing lines Patch % Lines
crates/bitcoin-da/src/service.rs 66.6% 6 Missing ⚠️
Additional details and impacted files
Files with missing lines Coverage Δ
crates/bitcoin-da/src/fee.rs 96.6% <100.0%> (+<0.1%) ⬆️
crates/bitcoin-da/src/service.rs 84.9% <66.6%> (-1.2%) ⬇️

... and 4 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

rakanalh
rakanalh previously approved these changes Sep 30, 2025
Copy link
Contributor

@rakanalh rakanalh left a comment

Choose a reason for hiding this comment

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

LGTM

kpp
kpp previously approved these changes Sep 30, 2025
@jfldde jfldde changed the title feat: Cap bitcoin-da fee rate feat: Temporary cap bitcoin-da fee rate Oct 1, 2025
@jfldde jfldde requested a review from eyusufatik October 3, 2025 07:58
@jfldde jfldde dismissed stale reviews from kpp and rakanalh via 23a4ee8 October 6, 2025 11:42
@jfldde jfldde mentioned this pull request Oct 10, 2025
25 tasks
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.

feat: batch proof sending stops over 15 sat/vbyte

4 participants