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

Check if doneChan is not nil. #9

Merged
merged 2 commits into from
Sep 26, 2013
Merged

Check if doneChan is not nil. #9

merged 2 commits into from
Sep 26, 2013

Conversation

crackcomm
Copy link
Contributor

No description provided.

@mreiferson
Copy link
Member

Specifically for the case when you want to "fire and forget" ala mongo :) ?

@crackcomm
Copy link
Contributor Author

Yes, because why not? :)

@mreiferson
Copy link
Member

Hah, I guess that's fine. Can you update each async method's godoc comment so that it explains that you can pass nil (and the effect it has)?

Thanks for your contribution.

@crackcomm
Copy link
Contributor Author

I hope it's enough ;)

mreiferson added a commit that referenced this pull request Sep 26, 2013
Check if doneChan is not nil.
@mreiferson mreiferson merged commit f148dc5 into nsqio:master Sep 26, 2013
amosir pushed a commit to amosir/go-nsq that referenced this pull request Jan 16, 2024
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.

2 participants