Skip to content
This repository has been archived by the owner on Mar 7, 2020. It is now read-only.

WIP: Reactive Wrapper for Synchronous Method Handler #18

Closed
wants to merge 11 commits into from

Conversation

kdavisk6
Copy link
Member

Work In Progress:

This is my proposal for an initial release of this library. I am aware that it is not "reactive all the way down", but it does provide a reasonable minimum viable product that adds the following:

  • Support for RxJava and Reactor Publisher return types
  • Maintains all Feign core features

Remaining Work

  • Test Cases for OkHttp
  • Test Cases for ApacheHttp
  • Test Cases for Request Interceptors
  • Test Cases for QueryMaps

@kptfh
Copy link
Collaborator

kptfh commented Sep 11, 2018

I'm afraid to be not polite but can you experiment on hybridizing reactive and sync feign in other repository and leave this one as is?

@kdavisk6 kdavisk6 closed this Oct 5, 2018
@kdavisk6 kdavisk6 deleted the reactive-core branch October 5, 2018 18:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants