There are too many places where we have very inconsistent return types involving promises. Particularly cases where functions throw, return a resolving promise, and also return a rejecting promise. Also a lot of places where functions return promises that resolve with different value types in different paths.
HookRunner is one spot I've seen that's particularly bad for this, but I've seen it in a few other spots too. My general recommendation: * Any function that returns a promise should never throw. It should reject the promise. * Any function that returns a promise should document the resolving value type of that promise, and ensure that only that value type is ever resolved. [ Full content available at: https://github.com/apache/cordova-lib/issues/714 ] This message was relayed via gitbox.apache.org for [email protected]
