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

refactor(samples): split query and table samples into separate files#384

Merged
JustinBeckwith merged 10 commits intogoogleapis:masterfrom
steffnay:update-to-snippet-rubric
Mar 15, 2019
Merged

refactor(samples): split query and table samples into separate files#384
JustinBeckwith merged 10 commits intogoogleapis:masterfrom
steffnay:update-to-snippet-rubric

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 15, 2019
@steffnay steffnay requested a review from jkwlui March 15, 2019 18:48
Comment thread README.md Outdated
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 15, 2019

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #384   +/-   ##
=======================================
  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 2383b70...be802d2. Read the comment docs.

@steffnay steffnay requested a review from jkwlui March 15, 2019 19:53
Copy link
Copy Markdown
Contributor

@JustinBeckwith JustinBeckwith left a comment

Choose a reason for hiding this comment

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

Can you please delete samples/default.profraw? Other than that, LGTM

@steffnay steffnay requested a review from JustinBeckwith March 15, 2019 21:45
@JustinBeckwith JustinBeckwith merged commit 8fcd1c9 into googleapis:master Mar 15, 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