Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update btcd dependency, add Signet params #735

Merged
merged 2 commits into from
Apr 28, 2021
Merged

Conversation

guggero
Copy link
Collaborator

@guggero guggero commented Mar 4, 2021

Required for lightningnetwork/lnd#5025.

Depends on btcsuite/btcd#1692.

Adds support for Bitcoin Signet.

cc @Roasbeef, @wpaulino.

Copy link
Contributor

@wpaulino wpaulino left a comment

Choose a reason for hiding this comment

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

LGTM, travis is failing with:

--- FAIL: TestRPCMethodHelpGeneration (0.01s)
    rpchelp_test.go:59: Cannot generate 'en_US' help for method 'listsinceblock': missing description for 'listtransactionsresult-label'
    rpchelp_test.go:59: Cannot generate 'en_US' help for method 'listtransactions': missing description for 'listtransactionsresult-label'
    rpchelp_test.go:59: Cannot generate 'en_US' help for method 'listaddresstransactions': missing description for 'listtransactionsresult-label'
    rpchelp_test.go:59: Cannot generate 'en_US' help for method 'listalltransactions': missing description for 'listtransactionsresult-label'
    rpchelp_test.go:46: Regenerate help texts with 'go generate' after fixing

@guggero
Copy link
Collaborator Author

guggero commented Mar 5, 2021

Fixed Travis.

@guggero
Copy link
Collaborator Author

guggero commented Apr 22, 2021

Rebased. @Roasbeef PTAL.

Copy link
Member

@Roasbeef Roasbeef left a comment

Choose a reason for hiding this comment

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

LGTM 🥃

go.mod Outdated Show resolved Hide resolved
With this commit we add the signet test network parameters to all switch
cases where we distinguish between the different networks.
@guggero
Copy link
Collaborator Author

guggero commented Apr 27, 2021

Rebased and added a few more cases that were previously not covered.

Copy link
Member

@Roasbeef Roasbeef 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 0b7de5e into btcsuite:master Apr 28, 2021
@guggero guggero deleted the signet branch April 28, 2021 06:44
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