-
Notifications
You must be signed in to change notification settings - Fork 35
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
fix(deps): update dependency @google-cloud/common to ^0.23.0 #281
Conversation
Breaking changes in common: common was split in multiple modules, HTTP dependency must be passed in by client library. Fixes: googleapis/nodejs-common#204 Fixes: googleapis#266
Codecov Report
@@ Coverage Diff @@
## master #281 +/- ##
=========================================
- Coverage 89.75% 89.7% -0.06%
=========================================
Files 6 6
Lines 410 408 -2
Branches 72 73 +1
=========================================
- Hits 368 366 -2
Misses 42 42
Continue to review full report at Codecov.
|
@fhinkel Looks like there are merge conflicts in ts/test/test-heap-profiler.ts? Also appveyor CI check is red, PTAL. |
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.
LGTM once CI is green.
super({ | ||
parent: new Service(serviceConfig, config), | ||
baseUrl: '/', | ||
requestModule: request |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
CI is green and merge ist resolved. Can somebody land this? (Or give me write access). Thanks |
Breaking changes in common: common was split in multiple modules, HTTP
dependency must be passed in by client library.
Fixes: googleapis/nodejs-common#204
Fixes: #266