Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Support kotlin coroutines #1706

Merged
merged 19 commits into from
Sep 14, 2022
Merged

Support kotlin coroutines #1706

merged 19 commits into from
Sep 14, 2022

Commits on Sep 8, 2022

  1. Support kotlin coroutines 

    Resolves: OpenFeign#1565

Inspired by PlaytikaOSS/feign-reactive#486

## TODO

- [ ] Separate Kotlin support module
- [ ] Enhance test case
- [ ] Refactoring
- [ ] Clean up pom.xml
    @wplong11
    wplong11 committed Sep 8, 2022
    Configuration menu
    Copy the full SHA
    50e3ce3 View commit details
    Browse the repository at this point in the history

  2. Apply optional dependency to kotlin support related dependency

    @wplong11
    wplong11 committed Sep 8, 2022
    Configuration menu
    Copy the full SHA
    4d31e1d View commit details
    Browse the repository at this point in the history

  3. Seperate Kotlin support module

    @wplong11
    wplong11 committed Sep 8, 2022
    Configuration menu
    Copy the full SHA
    593e0ad View commit details
    Browse the repository at this point in the history

  4. Remove unused code from ClassUtils.java

    @wplong11
    wplong11 committed Sep 8, 2022
    Configuration menu
    Copy the full SHA
    c039f5c View commit details
    Browse the repository at this point in the history

  5. Remove unused code from ClassUtils.java

    @wplong11
    wplong11 committed Sep 8, 2022
    Configuration menu
    Copy the full SHA
    93265fe View commit details
    Browse the repository at this point in the history

  6. Refactor KotlinDetector

    @wplong11
    wplong11 committed Sep 8, 2022
    Configuration menu
    Copy the full SHA
    3cd2ae1 View commit details
    Browse the repository at this point in the history

  7. Move ClassUtils location into KotlinDetector

    @wplong11
    wplong11 committed Sep 8, 2022
    Configuration menu
    Copy the full SHA
    115c849 View commit details
    Browse the repository at this point in the history

  8. Move KotlinDetector location

    @wplong11
    wplong11 committed Sep 8, 2022
    Configuration menu
    Copy the full SHA
    202b0f8 View commit details
    Browse the repository at this point in the history

  9. Format code

    @velo @wplong11
    velo authored and wplong11 committed Sep 8, 2022
    Configuration menu
    Copy the full SHA
    887d8f2 View commit details
    Browse the repository at this point in the history

  10. First attempt to move kotlin work to it's own isolated module

    @velo @wplong11
    velo authored and wplong11 committed Sep 8, 2022
    Configuration menu
    Copy the full SHA
    69f017e View commit details
    Browse the repository at this point in the history

  11. Coroutine Feign using AyncFeign

    @velo @wplong11
    velo authored and wplong11 committed Sep 8, 2022
    Configuration menu
    Copy the full SHA
    c48f1f5 View commit details
    Browse the repository at this point in the history

  12. Coroutine Feign using AyncFeign

    @velo @wplong11
    velo authored and wplong11 committed Sep 8, 2022
    Configuration menu
    Copy the full SHA
    e24c9af View commit details
    Browse the repository at this point in the history

  13. Refactor suspending function detect logic 

    - Remove KotlinDetector.java
- Add Method.isSuspend extension function
    @wplong11
    wplong11 committed Sep 8, 2022
    Configuration menu
    Copy the full SHA
    9e90725 View commit details
    Browse the repository at this point in the history

  14. Cleanup CoroutineFeignTest test code format

    @wplong11
    wplong11 committed Sep 8, 2022
    Configuration menu
    Copy the full SHA
    50162c7 View commit details
    Browse the repository at this point in the history

  15. Fix suspend function contract parsing error when using http body

    @wplong11
    wplong11 committed Sep 8, 2022
    Configuration menu
    Copy the full SHA
    6b65d79 View commit details
    Browse the repository at this point in the history

  16. Rename test names to be meaningful

    @wplong11
    wplong11 committed Sep 8, 2022
    Configuration menu
    Copy the full SHA
    a5fd11f View commit details
    Browse the repository at this point in the history

  17. Add Github Example With Coroutine 

    - Copy of GithubExample
    @wplong11
    wplong11 committed Sep 8, 2022
    Configuration menu
    Copy the full SHA
    b57140d View commit details
    Browse the repository at this point in the history

  18. Remove unnecessary dependency 

    https://github.com/OpenFeign/feign/pull/1706/files#r965389041
    @wplong11
    wplong11 committed Sep 8, 2022
    Configuration menu
    Copy the full SHA
    78589d5 View commit details
    Browse the repository at this point in the history

Commits on Sep 13, 2022

  1. Merge branch 'master' into 1565

    @velo
    velo committed Sep 13, 2022
    Configuration menu
    Copy the full SHA
    1e91c7a View commit details
    Browse the repository at this point in the history