Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
test: use arrow function instead of bind
Using an arrow function here eliminates the need to call `bind()`. PR-URL: #17202 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Evan Lucas <evanlucas@me.com> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
- Loading branch information
571ecd1
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.
@lance ... this is failing
make lint
on master$ make lint Running JS linter... /home/james/node/main/test/parallel/test-https-truncate.js 70:37 error Missing semicolon semi β 1 problem (1 error, 0 warnings) 1 error, 0 warnings potentially fixable with the `--fix` option.
571ecd1
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.
Fix: #17251
571ecd1
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.
@jasnell that is super embarrassing. Very sorry about that.
571ecd1
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.
No worries at all :-) ... I've done it more times than I care to admit!