aljaz/lightning
1
0
Fork 0
mirror of https://github.com/aljazceru/lightning.git synced 2025-12-24 09:34:24 +01:00
Code Issues Packages Projects Releases Wiki Activity

msggen: fix incorrect assertion.

Browse Source 
Adding a new field with `added` fails:

```
AssertionError: Field Feerates.perkb.estimates[] does not have an `added` annotation
```

Looks like this assertion is wrong: we should get an added from the field itself or
from the .msggen.json file.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
This commit is contained in:
Rusty Russell
2023-04-07 14:08:05 +09:30
parent ca80dee514
commit b53cc69cfd
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
contrib/msggen/msggen/patch.py
View File

@@ -67,7 +67,7 @@ class VersionAnnotationPatch(Patch):
added = m.get('added', None)
deprecated = m.get('deprecated', None)
assert added or not f.added, f"Field {f.path} does not have an `added` annotation"
assert added or f.added, f"Field {f.path} does not have an `added` annotation"
# We do not allow the added and deprecated flags to be
# modified after the fact.