fix end function when piping on request by aaharu · Pull Request #832 · forwardemail/supertest (original) (raw)
Navigation Menu
Provide feedback
Saved searches
Use saved searches to filter your results more quickly
Appearance settings
Closed
wants to merge 2 commits into
Conversation
Checklist
rework of #742
- I have ensured my pull request is not behind the main or master branch of the original repository.
- I have rebased all commits where necessary so that reviewing this pull request can be done without having to merge it first.
- I have written a commit message that passes commitlint linting.
- I have ensured that my code changes pass linting tests.
- I have ensured that my code changes pass unit tests.
- I have described my pull request and the reasons for code changes along with context if necessary.
Thomas and others added 2 commits
I'm also having this issue - thank you for the fix @aaharu
Would also love to see this PR go in! We're running into the issue with our repo!
| * @param {?Error} resError |
|---|
| * @param {Response} res |
| * @param {Function} fn |
| * @param {?Function} fn |
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please review your commit and fix these typos
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters
[ Show hidden characters]({{ revealButtonHref }})