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

refactor(samples): fix loadJSONFromGCSTruncate wrong function#386

Merged
steffnay merged 6 commits intogoogleapis:masterfrom
steffnay:fix-load-json-sample
Mar 17, 2019
Merged

refactor(samples): fix loadJSONFromGCSTruncate wrong function#386
steffnay merged 6 commits intogoogleapis:masterfrom
steffnay:fix-load-json-sample

Conversation

@steffnay
Copy link
Copy Markdown
Contributor

  • Tests and linter pass
  • Code coverage does not decrease (if any source code was changed)
  • Appropriate docs were updated (if necessary)

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Mar 17, 2019
@steffnay steffnay requested a review from jkwlui March 17, 2019 01:55
@steffnay steffnay requested a review from JustinBeckwith March 17, 2019 04:32
@JustinBeckwith
Copy link
Copy Markdown
Contributor

It looks like we introduced 404's in the docs with the previous samples change. Would you mind submitting a PR to address those first, so the CI goes green?

@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 17, 2019

Codecov Report

Merging #386 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #386   +/-   ##
=======================================
  Coverage   99.44%   99.44%           
=======================================
  Files           4        4           
  Lines         544      544           
  Branches       75       75           
=======================================
  Hits          541      541           
  Misses          2        2           
  Partials        1        1

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 8fcd1c9...179d430. Read the comment docs.

@steffnay steffnay merged commit cb44357 into googleapis:master Mar 17, 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.

4 participants