Skip to content

Conversation

@lukesneeringer
Copy link
Contributor

Fixes #3078.

@lukesneeringer lukesneeringer self-assigned this Feb 27, 2017
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Feb 27, 2017
Copy link
Contributor

@daspecster daspecster left a comment

Choose a reason for hiding this comment

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

LGTM pending Travis/Circle.

@lukesneeringer
Copy link
Contributor Author

This almost certainly is not okay as is -- the logging unit tests skipped tons of tests. Looking into it.

@lukesneeringer
Copy link
Contributor Author

I did not realize that I had not migrated this from .grpc. to .proto., and that was the issue when we threw it at the new GAPICs.

@daspecster
Copy link
Contributor

@lukesneeringer which tests were skipped?


setup(
name='google-cloud-logging',
version='0.23.0',

This comment was marked as spam.

This comment was marked as spam.

@lukesneeringer
Copy link
Contributor Author

@lukesneeringer which tests were skipped?

All the ones depending on GAX. I fixed it.

@lukesneeringer lukesneeringer merged commit 44c5575 into googleapis:master Feb 27, 2017
@lukesneeringer lukesneeringer deleted the issue-3078 branch February 27, 2017 23:27
richkadel pushed a commit to richkadel/google-cloud-python that referenced this pull request May 6, 2017
parthea pushed a commit that referenced this pull request Nov 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants