Skip to content
This repository was archived by the owner on Jan 21, 2026. It is now read-only.

fix: unpin @types/node and account for new http.request signatures#1120

Merged
kjin merged 1 commit intogoogleapis:masterfrom
kjin:types-node
Sep 30, 2019
Merged

fix: unpin @types/node and account for new http.request signatures#1120
kjin merged 1 commit intogoogleapis:masterfrom
kjin:types-node

Conversation

@kjin
Copy link
Contributor

@kjin kjin commented Sep 24, 2019

This PR adds support for a new signature of http.request in Node 10.9+. It also unpins @types/node.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Sep 24, 2019
@codecov
Copy link

codecov bot commented Sep 28, 2019

Codecov Report

Merging #1120 into master will increase coverage by <.01%.
The diff coverage is 91.11%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1120      +/-   ##
==========================================
+ Coverage   94.97%   94.98%   +<.01%     
==========================================
  Files          98       98              
  Lines        6451     6461      +10     
  Branches      500      503       +3     
==========================================
+ Hits         6127     6137      +10     
  Misses        166      166              
  Partials      158      158
Impacted Files Coverage Δ
test/test-cls-ah.ts 90.69% <0%> (ø) ⬆️
src/plugins/plugin-koa.ts 91.07% <100%> (ø) ⬆️
test/plugins/test-trace-http.ts 100% <100%> (ø) ⬆️
src/plugins/plugin-http.ts 91.34% <90.9%> (+0.16%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 41a92f7...1f03fbf. Read the comment docs.

@kjin kjin marked this pull request as ready for review September 28, 2019 02:04
@kjin kjin requested a review from a team September 28, 2019 02:04
Copy link

@hybrist hybrist left a comment

Choose a reason for hiding this comment

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

LGTM

@kjin kjin merged commit bd9863b into googleapis:master Sep 30, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants