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

docs: Fix example for bindCallback #7178

Merged
merged 1 commit into from
Feb 17, 2023
Merged

docs: Fix example for bindCallback #7178

merged 1 commit into from
Feb 17, 2023

Conversation

ondrasej
Copy link
Contributor

@ondrasej ondrasej commented Feb 4, 2023

docs: Fix the example for bindCallback.

Description:

The example, as it was, crashes with Error: cb is not a function on line 5. The new example demonstrates how arguments are forwarded or added.

Related issue (if exists):

The example, as it was, crashes with `Error: cb is not a function` on line 5.
@ondrasej ondrasej changed the title Fix example for bindCallback docs: Fix example for bindCallback Feb 4, 2023
Copy link
Member

@jakovljevic-mladen jakovljevic-mladen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks for fixing this.

@ondrasej
Copy link
Contributor Author

ondrasej commented Feb 6, 2023

Thanks for approving!

Is there something I need to do to merge, or just wait for someone who has merge rights?

@jakovljevic-mladen
Copy link
Member

or just wait for someone who has merge rights?

Yeah, we have to wait for the project team lead to do the merge.

@benlesh benlesh merged commit fc3dab5 into ReactiveX:master Feb 17, 2023
benlesh pushed a commit that referenced this pull request Feb 17, 2023
The example, as it was, crashes with `Error: cb is not a function` on line 5.
@benlesh
Copy link
Member

benlesh commented Feb 17, 2023

Merged into 7.x and master.

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

Successfully merging this pull request may close these issues.

3 participants