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.

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

pyln spec update #5621

Merged
merged 2 commits into from
Sep 22, 2022
Merged

Conversation

rustyrussell
Copy link
Contributor

This doesn't really belong in the same repo as CLN, but it's where it is. So I'll publish this as soon as it's acked, to fix lnprototest master which uses the new field names!

1. Update default specdir to the new modern name.
2. Don't use python to extract version, use sed.
3. Use poetry publish.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
In particular, some field names have changed. lnprototest in master
has already upgraded names, so this brings us into compatibility again.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Changelog-Changed: pyln-spec: package updated to latest spec version.
@cdecker
Copy link
Member

cdecker commented Sep 22, 2022

Ignoring the lnprototest CI configs, as that's due to the chicken-and-egg nature of this PR :-)

@cdecker
Copy link
Member

cdecker commented Sep 22, 2022

ACK 22906c5

@cdecker cdecker merged commit 264b4e0 into ElementsProject:master Sep 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants