-
Notifications
You must be signed in to change notification settings - Fork 29.8k
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
Fixes: https://github.com/nodejs/node/issues/38606 #38610
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
kindly move the section writeStream.close([callback])
below writeStream.bytesWritten
to maintain the ASCII order based on headings
Hi, thank you for your comments. Please see if everything looks okay. This is my first ever issue that I have contributed to. So any other pointers or direction that I should keep in mind going forward contributing to other issues will be appreciated. :) |
Co-authored-by: Rich Trott <[email protected]>
PR-URL: #38610 Fixes: #38606 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Darshan Sen <[email protected]>
Landed in 5b90748 |
PR-URL: #38610 Fixes: #38606 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Darshan Sen <[email protected]>
PR-URL: #38610 Fixes: #38606 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Darshan Sen <[email protected]>
Fixes: #38606