-
Notifications
You must be signed in to change notification settings - Fork 43
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.0-beta.1 #185
Conversation
This is the same as v0.18.0-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.
LGTM 💯
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 a lot for the fixes! One request, otherwise looks good 🎉
tools/Dockerfile
Outdated
@@ -11,7 +11,7 @@ RUN cd /tmp \ | |||
&& mkdir -p /tmp/build/.cache \ | |||
&& mkdir -p /tmp/build/.modcache \ | |||
&& cd /tmp/tools \ | |||
&& go install -trimpath github.com/golangci/golangci-lint/cmd/golangci-lint \ | |||
&& go install -trimpath github.com/golangci/golangci-lint/cmd/golangci-lint@v1.59.0 \ |
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.
This should be updated in the go.mod
file in the tools
folder. That's the whole point of that go.mod
file in the first place, that we can pin version numbers of our tooling binaries we want to use.
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.
I see! Fixed it.
@@ -319,7 +319,7 @@ func TestGetLndInfo(t *testing.T) { | |||
require.Error(t, err) | |||
|
|||
// Error might be wrapped. | |||
require.True(t, errors.Is(err, test.expected)) | |||
require.ErrorIs(t, err, test.expected) |
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.
Nice 🔥
Update the image used to build to golang:1.22.3-bookworm. Updated the config: removed the linters that were removed, added new unconfigured linters creating a lot of noice, fixed one linter warning that made sense.
97107f5
to
81bbbea
Compare
This is the same as v0.18.0-beta.
Pull Request Checklist
in
lnd_services.go
are updated.macaroon_recipes.go
if your PR adds a new method that is calleddifferently than the RPC method it invokes.