Skip to content

btcutil: update to chaincfg/chainhash/v1.1.0#2072

Merged
Roasbeef merged 3 commits intobtcsuite:masterfrom
Roasbeef:chainhash-v-1-1-0-btcutil
Dec 19, 2023
Merged

btcutil: update to chaincfg/chainhash/v1.1.0#2072
Roasbeef merged 3 commits intobtcsuite:masterfrom
Roasbeef:chainhash-v-1-1-0-btcutil

Conversation

@Roasbeef
Copy link
Copy Markdown
Member

This is a follow up to #1978.

We bump to the new chainhash version, then start to use the functionality where applicable.

Next in the sequence is:

  1. Merge this.
  2. Update btcutil/psbt.
  3. Merge that
  4. Update the top-level pacakge (use the two new version above), then merge that.

We'll also want to revisit this PR, as it aims to resolve some of the current module issues that cause PR chains like this each time we update a sub-module: #1825

@Roasbeef Roasbeef force-pushed the chainhash-v-1-1-0-btcutil branch from 0685090 to 7745cbb Compare December 15, 2023 22:43
We also remove the replace directives in place.
@coveralls
Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 7227862106

  • 0 of 8 (0.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.003%) to 56.073%

Changes Missing Coverage Covered Lines Changed/Added Lines %
btcutil/gcs/builder/builder.go 0 8 0.0%
Totals Coverage Status
Change from base Build 7227596643: 0.003%
Covered Lines: 27982
Relevant Lines: 49903

💛 - Coveralls

@Roasbeef Roasbeef requested a review from guggero December 16, 2023 00:51
Copy link
Copy Markdown
Collaborator

@guggero guggero left a comment

Choose a reason for hiding this comment

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

LGTM 🎉

@Roasbeef Roasbeef merged commit 4c2ce6d into btcsuite:master Dec 19, 2023
asheswook pushed a commit to asheswook/btcd that referenced this pull request Apr 2, 2026
btcutil: update to chaincfg/chainhash/v1.1.0
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