Skip to content
This repository was archived by the owner on Nov 18, 2025. It is now read-only.

chore: document the way to override bundling options#847

Merged
gcf-merge-on-green[bot] merged 2 commits intomasterfrom
fix-doc
Jun 4, 2020
Merged

chore: document the way to override bundling options#847
gcf-merge-on-green[bot] merged 2 commits intomasterfrom
fix-doc

Conversation

@xiaozhenliu-gg5
Copy link
Contributor

@xiaozhenliu-gg5 xiaozhenliu-gg5 commented Jun 4, 2020

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jun 4, 2020
Copy link
Contributor

@alexander-fenster alexander-fenster left a comment

Choose a reason for hiding this comment

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

This is OK for documenting on the google-gax side. I think the original issue asks for some extra documentation in @google-cloud/logging, can we add something about redefining bundling to jsdoc there?

@codecov
Copy link

codecov bot commented Jun 4, 2020

Codecov Report

Merging #847 into master will decrease coverage by 0.04%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     googleapis/gax-nodejs#847      +/-   ##
==========================================
- Coverage   89.76%   89.71%   -0.05%     
==========================================
  Files          46       46              
  Lines        7442     7448       +6     
  Branches      516      516              
==========================================
+ Hits         6680     6682       +2     
- Misses        759      763       +4     
  Partials        3        3              
Impacted Files Coverage Δ
src/gax.ts 96.47% <100.00%> (+0.03%) ⬆️
.mocharc.js 78.57% <0.00%> (-14.29%) ⬇️

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 9a73bf9...6fad27e. Read the comment docs.

@alexander-fenster alexander-fenster added the automerge Merge the pull request once unit tests and other checks pass. label Jun 4, 2020
@gcf-merge-on-green gcf-merge-on-green bot merged commit 77f16fd into master Jun 4, 2020
@xiaozhenliu-gg5 xiaozhenliu-gg5 deleted the fix-doc branch June 4, 2020 18:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

automerge Merge the pull request once unit tests and other checks pass. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add bundling config overridability to CallOptions

3 participants