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

3.x: Change the parameter order of concatMap{Single|Maybe|Completable}DelayError #6610

Closed
akarnokd opened this issue Aug 5, 2019 · 2 comments
Closed

3.x: Change the parameter order of concatMap{Single|Maybe|Completable}DelayError #6610

akarnokd opened this issue Aug 5, 2019 · 2 comments
Labels
3.x Cleanup
Milestone
3.0

Comments

@akarnokd
Copy link
Member

akarnokd commented Aug 5, 2019

The regular concatMapDelayError uses parameter order of mapper, prefetch, tillTheEnd but the S/M/C variants have mapper, tillTheEnd, prefetch. The parameter ordering should match the original concatMapDelayError.
@akarnokd akarnokd added this to the 3.0 milestone Aug 5, 2019
@akarnokd
Copy link
Member Author

akarnokd commented Aug 5, 2019

This is not open to PR yet because I'm working on the rest of #6587

@akarnokd
Copy link
Member Author

Fixed via #6638.

@akarnokd akarnokd mentioned this issue Aug 28, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
3.x Cleanup
Projects
None yet
Milestone
3.0
Development

No branches or pull requests
1 participant
@akarnokd