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

fix: handle parameters to image.create() #541

Merged
merged 1 commit into from
Jan 27, 2021

Conversation

stephenplusplus
Copy link
Contributor

Fixes #540

image.create() accepts one too many parameters for what ServiceObject.create() was designed to support. This PR adds a new create() method to bypass it.

@stephenplusplus stephenplusplus requested a review from a team as a code owner January 20, 2021 17:17
@product-auto-label product-auto-label bot added the api: compute Issues related to the googleapis/nodejs-compute API. label Jan 20, 2021
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Jan 20, 2021
@codecov
Copy link

codecov bot commented Jan 20, 2021

Codecov Report

Merging #541 (330285a) into master (d1d9f4c) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #541   +/-   ##
=======================================
  Coverage   99.61%   99.61%           
=======================================
  Files          22       22           
  Lines       11944    11948    +4     
=======================================
+ Hits        11898    11902    +4     
  Misses         46       46           
Impacted Files Coverage Δ
src/image.js 100.00% <100.00%> (ø)
src/index.js 100.00% <100.00%> (ø)

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 d1d9f4c...330285a. Read the comment docs.

Copy link

@cschuff cschuff left a comment

Choose a reason for hiding this comment

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

LGTM 🌈

@bcoe bcoe merged commit 3b3a7dd into googleapis:master Jan 27, 2021
gcf-merge-on-green bot pushed a commit that referenced this pull request Jan 27, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
api: compute Issues related to the googleapis/nodejs-compute API. cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Image.create options are not passed correctly to Compute.createImage
3 participants