Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

build: credential-file-override is no longer required #1029

Merged
merged 2 commits into from
Aug 12, 2020

Conversation

yoshi-automation
Copy link
Contributor

This PR was generated using Autosynth. 🌈

Synth log will be available here:
https://source.cloud.google.com/results/invocations/5f7f9c6d-c75a-4c60-8bb8-0026a14cead7/targets

  • To automatically regenerate this PR, check this box.

Source-Link: googleapis/synthtool@94421c4

@chingor13 has configured our runtime environment, such that we no longer need to explicitly provide a credentials file.

How does this magic work, I didn't fully understand?

Source-Author: Benjamin E. Coe <bencoe@google.com>
Source-Date: Fri Aug 7 19:02:09 2020 -0700
Source-Repo: googleapis/synthtool
Source-Sha: 94421c47802f56a44c320257b2b4c190dc7d6b68
Source-Link: googleapis/synthtool@94421c4
@yoshi-automation yoshi-automation requested a review from a team as a code owner August 11, 2020 11:29
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Aug 11, 2020
@codecov
Copy link

codecov bot commented Aug 11, 2020

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1029   +/-   ##
=======================================
  Coverage   91.52%   91.52%           
=======================================
  Files          21       21           
  Lines        4093     4093           
  Branches      488      488           
=======================================
  Hits         3746     3746           
  Misses        347      347           

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 6dfee1d...d4c326b. Read the comment docs.

@sofisl sofisl changed the title build: --credential-file-override is no longer required build: credential-file-override is no longer required Aug 12, 2020
@sofisl sofisl added the automerge Merge the pull request once unit tests and other checks pass. label Aug 12, 2020
@gcf-merge-on-green gcf-merge-on-green bot merged commit b976c8a into master Aug 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Merge the pull request once unit tests and other checks pass. cla: yes This human has signed the Contributor License Agreement. context: full
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants