Skip to content

Conversation

@thunderbiscuit
Copy link
Member

@thunderbiscuit thunderbiscuit commented Oct 18, 2024

Removed our TxIn in favour of the bitcoin-ffi one. Note that bitcoindevkit/bitcoin-ffi#25 must be merged first, and this commit updated to use an official release of the library instead of the commit of the PR. Do not merge until bitcoin-ffi 0.1.3 is released.

Changelog notice

None.

Checklists

All Submissions:

  • I've signed all my commits
  • I followed the contribution guidelines
  • I ran cargo fmt and cargo clippy before committing

@thunderbiscuit thunderbiscuit force-pushed the refactor/txin branch 3 times, most recently from a77f9e8 to cd75c64 Compare October 21, 2024 15:31
@reez
Copy link
Collaborator

reez commented Feb 1, 2025

Reviewing bitcoindevkit/bitcoin-ffi#29 and having this merged into bdk-ffi to use the bitcoin-ffi TxIn type would be helpful with finishing out testing the integration (can explain more on bindings call Tuesday).

I see you'll have to fix conflicts, but bitcoindevkit/bitcoin-ffi#25 has been merged in so that's not a blocker anymore at least it looks like

@thunderbiscuit
Copy link
Member Author

This is not required anymore.

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.

2 participants