Improve `isReturnTypeList` for non suspend method

[theapache64]

Improve `isReturnTypeList` for non suspend method

[tuanchauict]

Another approach is to stop supporting too many types. We can limit to only suspend or normal function which returns List or Collection.

IMO, if a dev wants to support Flow for API, he/she can easily create a repository class. The API just does request and response data, not handle the other things (Single responsibility principle).

I definitely vote for this.
@theapache64 What do you think?

[theapache64]

@tuanchauict

Sorry for the late reply. I somehow missed this issue.

Incorrect if the return type is something like this: Foo<Bar<Baz<List>>

As of now, technically, retrosheet can only return an Object or List<Object>. so the above given generic wouldn't be technically possible at all. This means, even if we write support for that, that case won't occur.

If we extract secondary type from Flow like what ...

If the consumer wants to use Flow, Resource, or any other custom type. They'll have to write custom retrofit call adapters for that. I think this doesn't come under retrosheet.