Skip to content
This repository was archived by the owner on Mar 4, 2026. It is now read-only.

Remove autoCreate for objects that return operations.#133

Merged
stephenplusplus merged 4 commits intogoogleapis:masterfrom
stephenplusplus:spp--127
Mar 2, 2018
Merged

Remove autoCreate for objects that return operations.#133
stephenplusplus merged 4 commits intogoogleapis:masterfrom
stephenplusplus:spp--127

Conversation

@stephenplusplus
Copy link
Copy Markdown
Contributor

@stephenplusplus stephenplusplus commented Feb 28, 2018

Fixes #127

This adds get-or-create behavior to database.get() and instance.get().

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Feb 28, 2018
@ghost ghost assigned stephenplusplus Feb 28, 2018
@stephenplusplus
Copy link
Copy Markdown
Contributor Author

Please discuss over in #127 if you think this is the wrong solution.

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 1, 2018

Codecov Report

Merging #133 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff          @@
##           master   #133   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          12     12           
  Lines        1355   1393   +38     
=====================================
+ Hits         1355   1393   +38
Impacted Files Coverage Δ
src/database.js 100% <100%> (ø) ⬆️
src/instance.js 100% <100%> (ø) ⬆️

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 5449593...c192f43. Read the comment docs.

@stephenplusplus
Copy link
Copy Markdown
Contributor Author

@callmehiphop ready for review.

@stephenplusplus stephenplusplus changed the title Remove autoCreate for obejcts that return operations. Remove autoCreate for objects that return operations. Mar 2, 2018
@stephenplusplus stephenplusplus merged commit 495574d into googleapis:master Mar 2, 2018
@ghost ghost removed the cla: yes This human has signed the Contributor License Agreement. label Mar 2, 2018
@stephenplusplus stephenplusplus deleted the spp--127 branch March 2, 2018 16:26
@stephenplusplus stephenplusplus mentioned this pull request Mar 8, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants