Skip to content
This repository has been archived by the owner on Aug 28, 2024. It is now read-only.

fix: bitcoin lib final fixes #14

Merged
merged 12 commits into from
Aug 22, 2024
Merged

Conversation

steph-rs
Copy link
Collaborator

What ❔

Why ❔

Checklist

  • PR title corresponds to the body of PR (we generate changelog entries from PRs).
  • Tests for the changes have been added / updated.
  • Documentation comments have been added / updated.
  • Code has been formatted via zk fmt and zk lint.

@steph-rs steph-rs marked this pull request as ready for review August 20, 2024 17:28
Copy link
Collaborator

@mpavlovic-txfusion mpavlovic-txfusion left a comment

Choose a reason for hiding this comment

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

Marked few things to improve.

@steph-rs
Copy link
Collaborator Author

In the next commits I will resolve conflicts with base branch and fix the regtest module, including feature

…ation' into bitcoin-lib-fixes

# Conflicts:
#	core/lib/via_btc_client/src/client/mod.rs
#	core/lib/via_btc_client/src/regtest.rs
Copy link
Collaborator

@mpavlovic-txfusion mpavlovic-txfusion left a comment

Choose a reason for hiding this comment

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

Looks great!

@irnb
Copy link
Collaborator

irnb commented Aug 22, 2024

I'm gonna have some conflict with this branch and also it's miss of new change on bitcoin client
but let's merge it and i'll rebase my branch with this branch and those new changes

LGTM

@irnb irnb merged commit 0b4eea4 into feat/bitcoin-integration Aug 22, 2024
2 checks passed
@irnb irnb deleted the bitcoin-lib-fixes branch August 22, 2024 11:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants