The Wayback Machine - https://web.archive.org/web/20220322214815/https://github.com/nodejs/node/pull/33551
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.

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

http: simplify Agent initialization #33551

Closed
wants to merge 1 commit into from

Conversation

Himself65
Copy link
Member

@Himself65 Himself65 commented May 25, 2020

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines

@nodejs-github-bot nodejs-github-bot added the http label May 25, 2020
MrHeer
MrHeer approved these changes May 25, 2020
@nodejs-github-bot
Copy link
Contributor

@nodejs-github-bot nodejs-github-bot commented May 25, 2020

CI: https://ci.nodejs.org/job/node-test-pull-request/31550/ 💛 (known flake: parallel/test-http2-reset-flood)

@BridgeAR BridgeAR added the author ready label May 25, 2020
lpinca
lpinca approved these changes May 25, 2020
@lpinca
Copy link
Member

@lpinca lpinca commented May 25, 2020

Subsystem in commit message should be http: not lib:

@Himself65 Himself65 self-assigned this May 27, 2020
@Himself65 Himself65 changed the title lib: simplify Agent initialization http: simplify Agent initialization May 27, 2020
ronag
ronag approved these changes May 27, 2020
@nodejs-github-bot
Copy link
Contributor

@nodejs-github-bot nodejs-github-bot commented May 28, 2020

@nodejs-github-bot
Copy link
Contributor

@nodejs-github-bot nodejs-github-bot commented May 28, 2020

Himself65 added a commit that referenced this issue May 28, 2020
PR-URL: #33551
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Robert Nagy <ronagy@icloud.com>
@Himself65
Copy link
Member Author

@Himself65 Himself65 commented May 28, 2020

landed in f19a576

@Himself65 Himself65 closed this May 28, 2020
@Himself65 Himself65 deleted the 20200525-refactor branch May 28, 2020
codebytere added a commit that referenced this issue Jun 18, 2020
PR-URL: #33551
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Robert Nagy <ronagy@icloud.com>
@codebytere codebytere mentioned this pull request Jun 28, 2020
codebytere added a commit that referenced this issue Jun 30, 2020
PR-URL: #33551
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Robert Nagy <ronagy@icloud.com>
@codebytere codebytere added the backport-blocked-v12.x label Jul 8, 2020
@codebytere
Copy link
Member

@codebytere codebytere commented Jul 8, 2020

Blocked on v12.x for the same reason #32801 is blocked.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
author ready backport-blocked-v12.x http
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

10 participants
Morty Proxy This is a proxified and sanitized view of the page, visit original site.