Skip to content
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.

Sign up for GitHub

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

Jump to bottom

pyln: remove unused variable #6242

Merged
merged 1 commit into from
May 9, 2023
Merged

pyln: remove unused variable #6242

merged 1 commit into from
May 9, 2023

Conversation

morehouse
Copy link
Contributor

Variable inroduced in 708fb17 is making CI fail.

@morehouse
pyln: remove unused variable
e39893e 
Variable inroduced in 708fb17 is making CI fail.
@morehouse morehouse requested a review from cdecker as a code owner May 8, 2023 14:50
vincenzopalazzo
vincenzopalazzo approved these changes May 8, 2023
View reviewed changes
Copy link
Contributor

@vincenzopalazzo vincenzopalazzo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK e39893e

@endothermicdev
Copy link
Collaborator

ACK e39893e

@endothermicdev endothermicdev mentioned this pull request May 9, 2023
Merged
@ShahanaFarooqui ShahanaFarooqui merged commit fe5f3ce into ElementsProject:master May 9, 2023
@morehouse morehouse deleted the fix_pyln_utils branch May 9, 2023 20:45
@ShahanaFarooqui ShahanaFarooqui added this to the v23.05 milestone May 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vincenzopalazzo vincenzopalazzo vincenzopalazzo approved these changes

@cdecker cdecker Awaiting requested review from cdecker cdecker is a code owner

Assignees
No one assigned
Labels
pyln testing
Projects
None yet
Milestone
v23.05
Development

Successfully merging this pull request may close these issues.
4 participants
@morehouse @endothermicdev @vincenzopalazzo @ShahanaFarooqui