Skip to content

Conversation

@GideonBature
Copy link
Contributor

@GideonBature GideonBature commented Oct 7, 2025

Description

Renames cancel_tx() to unreserve_change_address() to better reflect its current functionality, which only frees up the change address and does not yet handle UTXO reservation.

Changelog notice

Changed

  • Renamed cancel_tx() to unreserve_change_address() for clarity

Closes #212

@luisschwab luisschwab moved this to In Progress in BDK Wallet Oct 7, 2025
@luisschwab luisschwab moved this from In Progress to Needs Review in BDK Wallet Oct 7, 2025
@GideonBature GideonBature marked this pull request as draft October 7, 2025 07:07
@GideonBature GideonBature force-pushed the freeing-reserved-utxos branch from e5b8d34 to 3f7ff2d Compare October 7, 2025 19:07
@GideonBature GideonBature marked this pull request as ready for review October 7, 2025 19:08
@coveralls
Copy link

Pull Request Test Coverage Report for Build 18323393969

Details

  • 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 84.813%

Totals Coverage Status
Change from base Build 18318022693: 0.0%
Covered Lines: 6908
Relevant Lines: 8145

💛 - Coveralls

@GideonBature GideonBature changed the title Implement freeing up reserved utxos and cancel_tx Rename cancel_tx to unreserve_change_address Oct 7, 2025
@ValuedMammal ValuedMammal added this to the Wallet 3.0.0 milestone Oct 27, 2025
@luisschwab
Copy link
Member

@GideonBature can you reword the commit as fix!: rename cancel_tx to unreserve_change_address to align with functionality to denote this is a breaking change?

@GideonBature
Copy link
Contributor Author

Noted @luisschwab let me do that now.

@GideonBature GideonBature force-pushed the freeing-reserved-utxos branch from 3f7ff2d to 35faa29 Compare December 4, 2025 11:22
@GideonBature
Copy link
Contributor Author

This recent push reword the commit message.

@codecov
Copy link

codecov bot commented Dec 4, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (master@321c8a0). Learn more about missing BASE report.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #326   +/-   ##
=========================================
  Coverage          ?   85.24%           
=========================================
  Files             ?       23           
  Lines             ?     8229           
  Branches          ?        0           
=========================================
  Hits              ?     7015           
  Misses            ?     1214           
  Partials          ?        0           
Flag Coverage Δ
rust 85.24% <100.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Needs Review

Development

Successfully merging this pull request may close these issues.

Rename cancel_tx to unreserve_change_address

4 participants