-
Notifications
You must be signed in to change notification settings - Fork 208
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
feat: accept apiEndpoint override #455
Conversation
BREAKING CHANGE: This adds the apiEndpoint property as a required field for the ServiceConfig, and makes it a public property on the Service class. This is being added to broadly support apiEndpoint overrides. See googleapis/nodejs-bigquery#455 to see how this would be used.
@bcoe @stephenplusplus this is ready for a real look. |
Codecov Report
@@ Coverage Diff @@
## master #455 +/- ##
==========================================
+ Coverage 99.42% 99.42% +<.01%
==========================================
Files 5 5
Lines 699 700 +1
Branches 194 194
==========================================
+ Hits 695 696 +1
- Misses 2 3 +1
+ Partials 2 1 -1
Continue to review full report at Codecov.
|
1 similar comment
Codecov Report
@@ Coverage Diff @@
## master #455 +/- ##
==========================================
+ Coverage 99.42% 99.42% +<.01%
==========================================
Files 5 5
Lines 699 700 +1
Branches 194 194
==========================================
+ Hits 695 696 +1
- Misses 2 3 +1
+ Partials 2 1 -1
Continue to review full report at Codecov.
|
BREAKING CHANGE: This adds the apiEndpoint property as a required field for the ServiceConfig, and makes it a public property on the Service class. This is being added to broadly support apiEndpoint overrides. See googleapis/nodejs-bigquery#455 to see how this would be used.
This lets developers provide an api endpoint to override the backend service Url.