Skip to content

Conversation

@guggero
Copy link
Contributor

@guggero guggero commented Dec 23, 2021

Prepare for new client release by bumping the version (and fixing a linter issue left over from #330).

Pull Request Checklist

  • LndServices minimum version has been updated if new lnd apis/fields are
    used.

@guggero
Copy link
Contributor Author

guggero commented Dec 23, 2021

Something's up with the itests when using this version of the client, need to investigate before merging and releasing this.

@lightninglabs-deploy
Copy link
Collaborator

@positiveblue: review reminder

@guggero
Copy link
Contributor Author

guggero commented Jan 4, 2022

I found the bug in the itest and fixed it (both client side and server side). This PR is ready for final review now.

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 2df6d90 into master Jan 5, 2022
@guggero guggero deleted the pool-0.5.4 branch January 6, 2022 08:50
positiveblue pushed a commit to positiveblue/pool that referenced this pull request Oct 11, 2022
…t-trader

rpcserver: make check + add trader operations atomic
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.

5 participants