Skip to content

Conversation

@theacodes
Copy link
Contributor

Resolves #4068

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Oct 19, 2017
@theacodes
Copy link
Contributor Author

@lukesneeringer @dhermes this is my attempt to fix #4068 without making myself angry. I'm open to feedback and suggestions at alternatives.

Args:
python_version (str): The Python interpreter version, for example,
``'2.7.13'``.
grpc_version (st): The gRPC library version.

This comment was marked as spam.

This comment was marked as spam.

``'2.7.13'``.
grpc_version (st): The gRPC library version.
api_core_version (str): The google-api-core library version.
gapic_version (Optional[str]): The version of gapic used to generate

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.

LGTM other than the two nits. I can fix the nits if you like (I will in a few minutes if you do not get to it first.)

@theacodes theacodes merged commit c2d496a into googleapis:master Oct 20, 2017
@theacodes theacodes deleted the api-core-metrics branch October 20, 2017 19:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants