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

Change res.flush to accept a callback function argument #57

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Change res.flush to accept a callback function argument #57

wants to merge 1 commit into from

Conversation

bijoythomas
Copy link

Discussions around this can be found at #56

@dougwilson dougwilson self-assigned this Oct 25, 2015
@dougwilson
Copy link
Contributor

It works on all supported Node.js versions, so seems fine to me 👍

@bijoythomas
Copy link
Author

Hey Doug, is there any timeline or module release date when this will be included?

@usernameisalreadytaken2014 usernameisalreadytaken2014 mentioned this pull request Feb 29, 2016
Closed
@dougwilson dougwilson force-pushed the master branch 3 times, most recently from d7bb81b to cd957aa Compare May 30, 2018 04:09
@bjohansebas
Copy link
Member

Hi @bijoythomas, this PR has conflicts, please resolve them.

@bjohansebas bjohansebas removed the pr label Nov 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bjohansebas bjohansebas bjohansebas approved these changes

Assignees

@dougwilson dougwilson

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@bijoythomas @dougwilson @bjohansebas