endo icon indicating copy to clipboard operation
endo copied to clipboard

`E.sendOnly(e).foo(args)`, etc, should have void return type

Open erights opened this issue 1 year ago • 1 comments

https://github.com/endojs/endo/blob/7a2955080f34880c8028e1b2dff06bf7b969790f/packages/eventual-send/src/E.js#L299

says it returns a promise for void. But does it actually? AFAICT, it immediately returns only undefined and therefore should be declared as returning either void or undefined.

I noticed this when I got a @typescript-eslint/no-floating-promises lint error on the statement

 E.sendOnly(guestPingee).ping('sendOnly');

erights avatar Jun 27 '24 18:06 erights

@michaelfig , same problem at https://github.com/Agoric/agoric-sdk/blob/cbe061cdb847d99ea435c26131709527b40a1ce5/packages/vow/src/E.js#L354 ?

erights avatar Jun 27 '24 19:06 erights