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

Check in synth script and update gax dependency #62

Merged
merged 5 commits into from
Jun 28, 2018

Conversation

jkwlui
Copy link
Member

@jkwlui jkwlui commented Jun 26, 2018

Fixes #59
ref: googleapis/google-cloud-node#2837

  • 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 Jun 26, 2018
@ghost ghost assigned jkwlui Jun 26, 2018
@jkwlui jkwlui requested a review from a team June 26, 2018 22:41
@codecov
Copy link

codecov bot commented Jun 26, 2018

Codecov Report

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

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #62   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           1      1           
  Lines           4      4           
=====================================
  Hits            4      4

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 e7abbc6...13d552e. Read the comment docs.

synth.py Outdated
for version in versions:
library = gapic.node_library(
'monitoring', version,
config_path="/google/monitoring/"

This comment was marked as spam.

This comment was marked as spam.

synth.py Outdated
#
subprocess.run(['npm', 'install'])

# # prettify and lint

This comment was marked as spam.

@ghost ghost assigned JustinBeckwith Jun 27, 2018
@jkwlui jkwlui merged commit cbf5395 into googleapis:master Jun 28, 2018
@jkwlui jkwlui deleted the synth-check-in branch June 28, 2018 17:46
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