Skip to content

test:refactor test-https-set-timeout-server - #13032

Closed
Trott wants to merge 1 commit into
nodejs:masterfrom
Trott:refactor-https-timeout
Closed

test:refactor test-https-set-timeout-server#13032
Trott wants to merge 1 commit into
nodejs:masterfrom
Trott:refactor-https-timeout

Conversation

@Trott

Copy link
Copy Markdown
Member
  • Add common.mustCall() for error handlers.
  • Alphabetize modules
Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines
Affected core subsystem(s)

test https

* Add common.mustCall() for error handlers.
* Alphabetize modules
@TrottTrott added https Issues or PRs related to the https subsystem. test Issues and PRs related to the tests. labels May 15, 2017
@Trott

Copy link
Copy Markdown
MemberAuthor

@aqrlnaqrln left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The commit message should probably have a space after the colon.

Trott added a commit to Trott/io.js that referenced this pull request May 20, 2017
* Add common.mustCall() for error handlers.
* Alphabetize modules
PR-URL: nodejs#13032
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
@Trott

Copy link
Copy Markdown
MemberAuthor

Landed in 658741b (with a space added after the colon per @aqrln's comment).

@MylesBorins

Copy link
Copy Markdown
Contributor

This does not land cleanly in LTS. Please feel free to manually backport. Please also feel free to replace do-not-land if it is being backported

@Trott
Trott deleted the refactor-https-timeout branch January 13, 2022 22:45
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

httpsIssues or PRs related to the https subsystem.testIssues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

@Trott@MylesBorins@refack@jasnell@lpinca@cjihrig@aqrln