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

go.mod: update lnd to v0.18.3-beta #161

Merged
merged 1 commit into from
Sep 20, 2024

Conversation

starius
Copy link
Contributor

@starius starius commented Sep 20, 2024

Also updated versions of modules inside LND repo (lnd/sqldb etc) to the versions used by v0.18.3-beta.

Fixed few build errors caused by changes in APIs.

@starius starius marked this pull request as ready for review September 20, 2024 03:38
Copy link
Member

@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.

Looks good, thanks for the update. Just one comment about the command descriptions that need to be re-generated.

@@ -36,7 +36,7 @@ const (

// lndVersion is the current version of lnd that we support. This is
// shown in some commands that affect the database and its migrations.
lndVersion = "v0.18.0-beta"
lndVersion = "v0.18.3-beta"
Copy link
Member

Choose a reason for hiding this comment

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

We should run make docs after changing this value.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed!

Also updated the comments in go.mod and cmd/chantools/root.go adding the instruction to run make docs after changing LND version.

Also updated versions of modules inside LND repo (lnd/sqldb etc) to the versions
used by v0.18.3-beta.

Fixed few build errors caused by changes in APIs.

Ran "make docs" and updated the comments in go.mod and cmd/chantools/root.go
adding the instruction to run "make docs" after changing LND version.
Copy link
Member

@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.

Thanks, LGTM 🎉

// cmd/chantools/root.go.
github.com/lightningnetwork/lnd v0.18.0-beta.1
github.com/lightningnetwork/lnd/kvdb v1.4.8
// cmd/chantools/root.go and run "make docs".
Copy link
Member

Choose a reason for hiding this comment

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

👍

@@ -56,7 +56,7 @@ chantools sweeptimelockmanual \
--listchannels string channel input is in the format of lncli's listchannels format; specify '-' to read from stdin
--maxcsvlimit uint16 maximum CSV limit to use (default 2016)
--maxnumchanstotal uint16 maximum number of keys to try, set to maximum number of channels the local node potentially has or had (default 500)
--maxnumchanupdates uint maximum number of channel updates to try, set to maximum number of times the channel was used (default 500)
--maxnumchanupdates uint maximum number of channel updates to try, set to maximum number of times the channel was used (default 1000)
Copy link
Member

Choose a reason for hiding this comment

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

Looks like I forgot to update the docs a couple of times.

@guggero guggero merged commit 88a3a82 into lightninglabs:master Sep 20, 2024
2 checks passed
@starius starius deleted the update-lnd-v0.18.3-beta branch September 20, 2024 14: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.

2 participants