Skip to content
This repository has been archived by the owner on Jun 28, 2022. It is now read-only.

Python: use google.api.core.Operation for long running operations. #1509

Closed
wants to merge 4 commits into from

Conversation

landrito
Copy link
Contributor

@codecov-io
Copy link

codecov-io commented Aug 24, 2017

Codecov Report

Merging #1509 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #1509      +/-   ##
============================================
+ Coverage     86.92%   86.92%   +<.01%     
- Complexity     4112     4116       +4     
============================================
  Files           404      404              
  Lines         16575    16585      +10     
  Branches       1762     1764       +2     
============================================
+ Hits          14407    14417      +10     
  Misses         1683     1683              
  Partials        485      485
Impacted Files Coverage Δ Complexity Δ
...ansformer/py/PythonPackageMetadataTransformer.java 97.61% <100%> (+0.13%) 40 <3> (+4) ⬆️
...transformer/py/PythonImportSectionTransformer.java 96.84% <100%> (+0.01%) 51 <0> (ø) ⬇️

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 959d26e...fd98a1a. Read the comment docs.

@@ -11,6 +11,7 @@ import sys

install_requires = [
'google-auth>=1.0.2, <2.0dev',
'google-cloud-core>=0.27.0, <1.0dev',

This comment was marked as spam.

This comment was marked as spam.

Copy link
Contributor

@lukesneeringer lukesneeringer left a comment

Choose a reason for hiding this comment

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

I was not expecting you to do this -- thank you! One minor change. It is LGTM'd after that.

@@ -131,6 +131,9 @@ public boolean hasMissingDefaultOptions() {
@Nullable
public abstract String gapicPackageName();

@Nullable
public abstract List<String> operationTypes();

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

@landrito
Copy link
Contributor Author

This is now obsolete.

@landrito landrito closed this Nov 28, 2017
@landrito landrito deleted the python-new-operation branch September 24, 2018 19:46
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.

4 participants