Skip to content

update dependencies (LND v0.18.0-beta and co)#138

Merged
guggero merged 3 commits intolightninglabs:masterfrom
starius:update-lnd-v0.18.0-beta
Jun 18, 2024
Merged

update dependencies (LND v0.18.0-beta and co)#138
guggero merged 3 commits intolightninglabs:masterfrom
starius:update-lnd-v0.18.0-beta

Conversation

@starius
Copy link
Contributor

@starius starius commented Jun 2, 2024

Updated versions:

  • lnd: v0.17.4-beta -> v0.18.0-beta
  • lnd/kvdb: v1.4.4 -> v1.4.8
  • lnd/tor: v1.1.2 -> v1.1.3
  • lnd/healthcheck: v1.2.3 -> v1.2.4
  • lnd/tlv: v1.1.1 -> v1.2.6
  • neutrino: v0.16.0 -> v0.16.1-0.20240425105051-602843d34ffd
  • neutrino/cache: v1.1.1 -> v1.1.2
  • lndclient: v0.17.4-1 -> v0.18.0-2
  • loop: v0.26.6-beta -> v0.28.3
  • loop/swapserverrpc: v1.0.5 -> v1.0.8
  • pool: v0.6.2-beta.0.20230329135228-c3bffb52df3a -> v0.6.5-beta.0.20240531084722-4000ec802aaa
  • pool/auctioneerrpc: v1.0.7 -> v1.1.2
  • aperture: v0.1.21-beta.0.20230705004936-87bb996a4030 -> v0.3.2-beta
  • replace of protobuf: v1.30.0-hex-display -> v1.33.0-hex-display
  • Go: 1.21 -> 1.22.3

Pool version v0.6.5-beta.0.20240531084722-4000ec802aaa was used because it uses LND v0.18.0-beta, not v0.18.0-beta.rc3.

Fixed imports and API changes.

Updated Go version in README.

Updated LND version in cmd/chantools/root.go.
Ran make docs.


Updated golangci-ling, because its old version crashed trying to parse Go 1.22.3 version:

level=error msg="Running error: context loading failed: failed to load packages: failed to load with go/packages: err: exit status 1: stderr: go: errors parsing go.mod:\n/build/go.mod:3: invalid go version '1.22.3': must match format 1.23\n"

lint: update configs, golangci-lint to v1.59.0
Remove deprecated and deactivated linters from .golangci.yml.
Ignore too noisy linters: protogetter, depguard, mnd.
Fix linter warnings.
Use timeout instead of deadline in .golangci.yml.


test: don't modify test data

Copy a file to temp directory before opening it.

Fix #139

@starius starius force-pushed the update-lnd-v0.18.0-beta branch 4 times, most recently from 5fbb198 to e63a879 Compare June 3, 2024 01:52
@starius starius marked this pull request as ready for review June 3, 2024 01:56
@guggero guggero self-requested a review June 3, 2024 07:17
@starius starius force-pushed the update-lnd-v0.18.0-beta branch from e63a879 to beaac0d Compare June 3, 2024 15:37
@starius
Copy link
Contributor Author

starius commented Jun 3, 2024

Updated PR. Loop just released loop v0.28.3, using it instead of a non-tagged commit.

@starius starius force-pushed the update-lnd-v0.18.0-beta branch from beaac0d to a9f9808 Compare June 3, 2024 22:48
@starius
Copy link
Contributor Author

starius commented Jun 3, 2024

Updated PR. Updated LND to v0.18.0-beta.1.

@starius starius force-pushed the update-lnd-v0.18.0-beta branch from a9f9808 to 2fd1e8b Compare June 5, 2024 15:50
@starius
Copy link
Contributor Author

starius commented Jun 5, 2024

Updated PR.

Updated lndclient to v0.18.0-2.
Changed the way golangci-lint is updated: pinned the version in go.mod, not in Dockerfile
Fixed issue #139 (don't modify test data files in tests).

@lightninglabs-deploy
Copy link

@guggero: review reminder

Copy link
Collaborator

@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 a lot for the update!
Have one request, otherwise looks good to me 💯

Updated versions:

  - lnd: v0.17.4-beta -> v0.18.0-beta
  - lnd/kvdb: v1.4.4 -> v1.4.8
  - lnd/tor: v1.1.2 -> v1.1.3
  - lnd/healthcheck: v1.2.3 -> v1.2.4
  - lnd/tlv: v1.1.1 -> v1.2.6
  - neutrino: v0.16.0 -> v0.16.1-0.20240425105051-602843d34ffd
  - neutrino/cache: v1.1.1 -> v1.1.2
  - lndclient: v0.17.4-1 -> v0.18.0-2
  - loop: v0.26.6-beta -> v0.28.3
  - loop/swapserverrpc: v1.0.5 -> v1.0.8
  - pool: v0.6.2-beta.0.20230329135228-c3bffb52df3a ->
        v0.6.5-beta.0.20240531084722-4000ec802aaa
  - pool/auctioneerrpc: v1.0.7 -> v1.1.2
  - aperture: v0.1.21-beta.0.20230705004936-87bb996a4030 -> v0.3.2-beta
  - replace of protobuf: v1.30.0-hex-display -> v1.33.0-hex-display
  - Go: 1.21 -> 1.22.3

Pool version v0.6.5-beta.0.20240531084722-4000ec802aaa was used because it uses
LND v0.18.0-beta, not v0.18.0-beta.rc3.

Fixed imports and API changes.

Updated Go version in README.

Updated LND version in cmd/chantools/root.go.
Ran `make docs`.
@starius starius force-pushed the update-lnd-v0.18.0-beta branch from 2fd1e8b to 6ea680c Compare June 18, 2024 14:02
starius added 2 commits June 18, 2024 11:08
Remove deprecated and deactivated linters from .golangci.yml.

Ignore too noisy linters: protogetter, depguard, mnd.

Fix linter warnings.

Use timeout instead of deadline in .golangci.yml.
Copy a file to temp directory before opening it.

Fix lightninglabs#139
@starius starius force-pushed the update-lnd-v0.18.0-beta branch from 6ea680c to 65d6e52 Compare June 18, 2024 14:08
@starius starius requested a review from guggero June 18, 2024 14:22
Copy link
Collaborator

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

LGTM 🎉

@guggero guggero merged commit d173f8c into lightninglabs:master Jun 18, 2024
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.

Running unit tests changes wallet.db file committed into the repo

3 participants