Skip to content
This repository has been archived by the owner on Sep 19, 2023. It is now read-only.

fix: samples and tests updated #13

Merged
merged 3 commits into from
Aug 3, 2020
Merged

fix: samples and tests updated #13

merged 3 commits into from
Aug 3, 2020

Conversation

ikuleshov
Copy link
Contributor

Samples fixed and updated to use the correct client library.
Integration test added.

@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Jul 29, 2020
@codecov
Copy link

codecov bot commented Jul 29, 2020

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #13   +/-   ##
=======================================
  Coverage   98.20%   98.20%           
=======================================
  Files           5        5           
  Lines        6478     6478           
  Branches      241      241           
=======================================
  Hits         6362     6362           
  Misses        115      115           
  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 1b2cebf...528394b. Read the comment docs.

Copy link
Contributor

@bcoe bcoe left a comment

Choose a reason for hiding this comment

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

A couple notes, and looks like we could use a rebase.

samples/package.json Outdated Show resolved Hide resolved
samples/quickstart_installed_oauth2.js Show resolved Hide resolved
@ikuleshov ikuleshov requested a review from bcoe July 31, 2020 21:09
Samples fixed and updated to use the correct client library.
Integration test added.
@sofisl sofisl added kokoro:force-run Add this label to force Kokoro to re-run the tests. kokoro:run Add this label to force Kokoro to re-run the tests. labels Aug 3, 2020
@yoshi-kokoro yoshi-kokoro removed kokoro:run Add this label to force Kokoro to re-run the tests. kokoro:force-run Add this label to force Kokoro to re-run the tests. labels Aug 3, 2020
@sofisl sofisl added kokoro:force-run Add this label to force Kokoro to re-run the tests. kokoro:run Add this label to force Kokoro to re-run the tests. labels Aug 3, 2020
@yoshi-kokoro yoshi-kokoro removed kokoro:run Add this label to force Kokoro to re-run the tests. kokoro:force-run Add this label to force Kokoro to re-run the tests. labels Aug 3, 2020
@sofisl sofisl added kokoro:force-run Add this label to force Kokoro to re-run the tests. kokoro:run Add this label to force Kokoro to re-run the tests. labels Aug 3, 2020
@yoshi-kokoro yoshi-kokoro removed kokoro:run Add this label to force Kokoro to re-run the tests. kokoro:force-run Add this label to force Kokoro to re-run the tests. labels Aug 3, 2020
@sofisl sofisl added kokoro:force-run Add this label to force Kokoro to re-run the tests. kokoro:run Add this label to force Kokoro to re-run the tests. labels Aug 3, 2020
@yoshi-kokoro yoshi-kokoro removed kokoro:run Add this label to force Kokoro to re-run the tests. kokoro:force-run Add this label to force Kokoro to re-run the tests. labels Aug 3, 2020
@sofisl sofisl added kokoro:force-run Add this label to force Kokoro to re-run the tests. kokoro:run Add this label to force Kokoro to re-run the tests. labels Aug 3, 2020
@yoshi-kokoro yoshi-kokoro removed kokoro:run Add this label to force Kokoro to re-run the tests. kokoro:force-run Add this label to force Kokoro to re-run the tests. labels Aug 3, 2020
@sofisl sofisl closed this Aug 3, 2020
@sofisl sofisl reopened this Aug 3, 2020
Samples fixed and updated to use the correct client library.
Integration test added.
@bcoe
Copy link
Contributor

bcoe commented Aug 3, 2020

@ikuleshov thank you for the cleanup, this is looking great.

@ikuleshov ikuleshov merged commit 55f4366 into googleapis:master Aug 3, 2020
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.

None yet

4 participants