-
Notifications
You must be signed in to change notification settings - Fork 96
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
build: update to lnd v0.17.0 and latest version of Pool #633
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, just need to fix the sample data file because of the new memo
field in the channel list.
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.
The README needs to be updated. One of the CI checks is also failing. I also think we will wait till the final 0.17.0-beta is released before merging this
@Roasbeef, remember to re-request review from reviewers when ready |
Replaced by #645. |
We point to the rc for 0.17.0 for now. Also we had to update Pool as well, due to a replace directive inserted into lnd.