async keyword for functions in the Python API Client

[renepickhardt]

looking at the python API client I saw that the blocking functions are supposed to have an async keyword in order to return a future. c.f.:

lightning/contrib/pylightning/lightning/lightning.py

Line 89 in cf12130

are blocking, the corresponding python methods include an `async`

However I could not find any function that has this. so either delete the comment or add this to those functions

[cdecker]

We haven't been too strict about this yet, but we should definitely aim to backfill the ones that aren't there just yet.

[darosior]

Fixed by #2326 ?

[cdecker]

I don't think it is addressed by #2326 since that only deals with plugin side hooks and methods, whereas this issue is about the async argument to some of the client-side RPC methods which cause a future to be returned instead of blocking until the result is ready.

[djpnewton]

Am I missing something or should this issue still be open? I dont see any async function arguments

[vincenzopalazzo]

I think no, the pylightning is deprecated in favor of pyln-client

[djpnewton]

The comment still exists in pyln-client https://github.com/ElementsProject/lightning/blob/master/contrib/pyln-client/pyln/client/lightning.py#L424

[rustyrussell]

We can do async now, comment should be changed: not sure what it's trying to say!