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

Conversation

@georgiyekkert
Copy link
Contributor

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

  • Make sure to open an issue as a bug/issue before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
  • Ensure the tests and linter pass
  • Code coverage does not decrease (if any source code was changed)
  • Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> 🦕

@georgiyekkert georgiyekkert requested a review from a team as a code owner April 28, 2021 16:53
@product-auto-label product-auto-label bot added the api: compute Issues related to the googleapis/nodejs-compute API. label Apr 28, 2021
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Apr 28, 2021
@georgiyekkert
Copy link
Contributor Author

@alexander-fenster

@alexander-fenster alexander-fenster self-assigned this Apr 28, 2021
@alexander-fenster
Copy link
Contributor

alexander-fenster commented Apr 28, 2021

OK so for the enum values: can you use compute.protos.google.cloud.compute.v1.Operation.Status.DONE instead of the constant? To avoid typing that much:

const Status = compute.protos.google.cloud.compute.v1.Operation.Status;
// then use Status.DONE

All values can be found in protos/protos.d.ts.

@alexander-fenster alexander-fenster added the kokoro:run Add this label to force Kokoro to re-run the tests. label Apr 28, 2021
@codecov
Copy link

codecov bot commented Apr 28, 2021

Codecov Report

Merging #549 (d7d92e6) into diregapic (a739a73) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@              Coverage Diff              @@
##           diregapic     #549      +/-   ##
=============================================
- Coverage      97.17%   97.16%   -0.01%     
=============================================
  Files             77       77              
  Lines          78901    78901              
  Branches        3810     3737      -73     
=============================================
- Hits           76670    76666       -4     
- Misses          2158     2162       +4     
  Partials          73       73              
Impacted Files Coverage Δ
.mocharc.js 79.31% <100.00%> (ø)
src/v1/region_instance_groups_client.ts 96.42% <0.00%> (-0.27%) ⬇️
src/v1/images_client.ts 97.49% <0.00%> (-0.15%) ⬇️

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 a739a73...d7d92e6. Read the comment docs.

@alexander-fenster
Copy link
Contributor

To make lint pass: npx gts fix (can take a lot of time since the code is huge)

@alexander-fenster alexander-fenster merged commit fb6cde8 into googleapis:diregapic Apr 29, 2021
alexander-fenster pushed a commit that referenced this pull request Apr 29, 2021
* tests: add GCE tests for diregapic

* tests: review fixes
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. kokoro:run Add this label to force Kokoro to re-run the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants