-
Notifications
You must be signed in to change notification settings - Fork 34
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
Conversation
There was a problem hiding this 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" |
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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.
384ee41
to
223cc02
Compare
There was a problem hiding this 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". |
There was a problem hiding this comment.
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) |
There was a problem hiding this comment.
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.
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.