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.

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

Add a builder API for Composable #18

Closed
jbrisbin opened this issue May 15, 2013 · 0 comments
Closed

Add a builder API for Composable #18

jbrisbin opened this issue May 15, 2013 · 0 comments
Milestone

Comments

@jbrisbin
Copy link

Due to the recent constructor injection changes, it's not very easy to create a Composable that is based on an existing Reactor or that uses a specific Dispatcher implementation.

It would be nice to have a general-purpose builder API for Reactors and things that need Dispatchers so a Builder class that's independent is probably the best long-term way to go.

For the time being, though, I suggest we add a Builder inner class to Composable and have the static from method return this Builder just to provide a quick and easy alternative to what was previously possible by using the from method, then calling setDispatcher.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant