Skip to content
This repository has been archived by the owner on Mar 20, 2018. It is now read-only.

Remove references to grpc.beta #127

Merged
merged 2 commits into from
Sep 9, 2016
Merged

Conversation

geigerj
Copy link
Contributor

@geigerj geigerj commented Aug 23, 2016

Also bump dependency on grpc up to 1.0.0.

Fixes #126

Also bump dependency on grpc up to 1.0.0.
@codecov-io
Copy link

codecov-io commented Aug 23, 2016

Current coverage is 97.20% (diff: 100%)

Merging #127 into master will increase coverage by <.01%

@@             master       #127   diff @@
==========================================
  Files             8          8          
  Lines           607        608     +1   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits            590        591     +1   
  Misses           17         17          
  Partials          0          0          

Powered by Codecov. Last update d59635e...9b7c483

@@ -50,7 +50,7 @@

install_requires = [
'future>=0.15.2',
'grpcio>=1.0rc1',
'grpcio>=1.0.0',
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we need to regenerate the auto-gen code with 1.0.0 to ensure everything still works correctly?

Copy link
Contributor

Choose a reason for hiding this comment

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

Also, let's update test-requirements.txt.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Do we need to regenerate the auto-gen code with 1.0.0 to ensure everything still works correctly?

Yes. GAX 0.13.0 will not work with older auto-gen code, since those have only beta entry points into the gRPC stub.

Also, let's update test-requirements.txt.

Done

@bjwatson
Copy link
Contributor

bjwatson commented Sep 2, 2016

@geigerj FYI, I published a sample package of this at https://testpypi.python.org/pypi/google-gax

@bjwatson
Copy link
Contributor

bjwatson commented Sep 8, 2016

LGTM. Can you update https://pypi.python.org/pypi/google-gax? Should be almost identical to 0.13.0 on https://testpypi.python.org/pypi/google-gax.

@dhermes
Copy link
Contributor

dhermes commented Sep 8, 2016

Yay!

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