Skip to content

Conversation

sebaslogen
Copy link

No description provided.

@akarnokd
Copy link
Member

Great, thanks!

👍

@akarnokd akarnokd changed the title Fix Observable delay methods typos in documenation 1.x: Fix Observable delay methods typos in documenation Mar 23, 2016
@artem-zinnatullin
Copy link
Contributor

👍

@vanniktech
Copy link
Collaborator

@neoranga55 there's also one here

@sebaslogen
Copy link
Author

@vanniktech do you suggest adding the fix for Single to this current PR or opening a new one?

@akarnokd
Copy link
Member

Maybe its easier to search for that typo in all files and fix them in a single PR.

@vanniktech
Copy link
Collaborator

@akarnokd that's exactly what I did ack compuation and it returned me the two places he already fixed it and then the other one in Single

@akarnokd
Copy link
Member

Okay, to be clear, please fix the type on Single as well in this PR.

@sebaslogen
Copy link
Author

Ok, added the fix for Single, I'm not familiar with the branching flow here so I did the one I'm familiar with: rebase to collapse both changes into a single commit.

@vanniktech
Copy link
Collaborator

Looking good now 👍

@akarnokd
Copy link
Member

We prefer single commit PRs so you did right by squashing the commits.

@stevegury
Copy link
Member

👍

@akarnokd akarnokd merged commit f8be5c1 into ReactiveX:1.x Mar 23, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants