-
Notifications
You must be signed in to change notification settings - Fork 158
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 flatMapFirst #47
Comments
It's a great idea but would require and implementor with motivation that needs it. I don't have a use case for it so clearing my time to write it would be a bit tricky... |
What about this?
|
@troupmar Care to add that as a pull request? I use |
@danielt1263 Hi there, I will try to make a pull request but at this time I am very busy. The pull request will require some additional tests. At this point it would be useful if somebody could overview the implementation, at least to know if it seems to be correct. Then if I happen to have some time, I will be happy to complete it and prepare a pull request. |
Here it is: #119 |
While there's no
switchToFirst
operator in Combine, I find it very challenging to implementflatMapFirst
like operator from scratch, I'd suggest to start building this operator which's really useful for many use casesThe text was updated successfully, but these errors were encountered: