Skip to content
This repository was archived by the owner on Dec 23, 2024. It is now read-only.

fix: use compute.googleapis.com as base URI #168

Merged
merged 1 commit into from
Sep 6, 2019
Merged

Conversation

JustinBeckwith
Copy link
Contributor

No description provided.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Sep 6, 2019
@JustinBeckwith JustinBeckwith requested a review from bcoe September 6, 2019 18:00
@codecov
Copy link

codecov bot commented Sep 6, 2019

Codecov Report

Merging #168 into master will not change coverage.
The diff coverage is 66.66%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #168   +/-   ##
=======================================
  Coverage   48.43%   48.43%           
=======================================
  Files           1        1           
  Lines         351      351           
  Branches        6        6           
=======================================
  Hits          170      170           
  Misses        181      181
Impacted Files Coverage Δ
src/index.ts 48.43% <66.66%> (ø) ⬆️

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 56ccd07...d19481c. Read the comment docs.

@@ -17,7 +17,7 @@ images.getAll((err, images) => {
[
{
kind: 'compute#image',
selfLink: 'https://www.googleapis.com/compute/v1/projects/centos-cloud/global/images/centos-6-v20150710',
selfLink: 'https://compute.googleapis.com/compute/v1/projects/centos-cloud/global/images/centos-6-v20150710',
Copy link
Contributor

Choose a reason for hiding this comment

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

Does it really still have compute after the .com/?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It does 🙈

@JustinBeckwith JustinBeckwith merged commit 98c45e5 into master Sep 6, 2019
@stephenplusplus stephenplusplus deleted the split branch November 26, 2019 12:29
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