Skip to content

Conversation

@galt-tr
Copy link
Contributor

@galt-tr galt-tr commented Jul 2, 2025

This pull request updates the dependency from github.com/libsv/go-bt/v2 to github.com/bsv-blockchain/go-bt/v2 across multiple files in the project. The changes involve modifying the go.mod file to reflect the new dependency and updating import statements in various source files to use the new library.

Dependency Update:

  • go.mod: Replaced github.com/libsv/go-bt/v2 v2.2.5 with github.com/bsv-blockchain/go-bt/v2 v2.3.0 and removed the replace directive for github.com/libsv/go-bt/v2. [1] [2]

Import Statement Updates:

  • tx_map.go: Updated the import statement to use github.com/bsv-blockchain/go-bt/v2/chainhash instead of github.com/libsv/go-bt/v2/chainhash. [1] [2]
  • tx_map_benchmarks_test.go: Changed the chainhash import to reference github.com/bsv-blockchain/go-bt/v2/chainhash.
  • tx_map_fuzz_test.go: Updated the chainhash import to use github.com/bsv-blockchain/go-bt/v2/chainhash.
  • tx_map_test.go: Modified the chainhash import to point to github.com/bsv-blockchain/go-bt/v2/chainhash.

@galt-tr galt-tr requested a review from icellan as a code owner July 2, 2025 14:16
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jul 2, 2025

@codecov
Copy link

codecov bot commented Jul 2, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

📢 Thoughts on this report? Let us know!

@mrz1836 mrz1836 requested review from Copilot and mrz1836 July 2, 2025 14:20
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR migrates the project’s Go BT dependency from github.com/libsv/go-bt/v2 to github.com/bsv-blockchain/go-bt/v2.

  • Bump the go-bt module in go.mod to v2.3.0 and remove the old replace directive.
  • Update all import paths for chainhash in source and test files to point at the new module.

Reviewed Changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
go.mod Updated require for go-bt/v2 to v2.3.0 and removed old replace.
tx_map.go Updated comment and import to use new chainhash path.
tx_map_test.go Switched chainhash import to the new module path.
tx_map_fuzz_test.go Switched chainhash import to the new module path.
tx_map_benchmarks_test.go Switched chainhash import to the new module path.

@mrz1836 mrz1836 added the chore Simple dependency updates or version bumps label Jul 2, 2025
Copy link
Collaborator

@mrz1836 mrz1836 left a comment

Choose a reason for hiding this comment

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

LGTM

@mrz1836 mrz1836 added the refactor Any significant refactoring label Jul 2, 2025
@galt-tr galt-tr merged commit fa53b07 into master Jul 2, 2025
23 of 29 checks passed
@github-actions github-actions bot deleted the gobtUpdate branch July 2, 2025 14:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore Simple dependency updates or version bumps refactor Any significant refactoring

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants