Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: make versionClient a var #912

Merged
merged 3 commits into from Feb 23, 2022
Merged

Conversation

codyoss
Copy link
Member

@codyoss codyoss commented Feb 23, 2022

This will allow us to override the value after the client has been
generated in an init function. For that code please see
googleapis/google-cloud-go#5687

Previous "UNKNOWN" version behaviour in maintained in a new
function.

This will allow us to override the value after the client has been
generated in an init function. For that code please see
googleapis/google-cloud-go#5687
Copy link
Collaborator

@noahdietz noahdietz left a comment

Choose a reason for hiding this comment

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

LGTM with two nits

internal/gengapic/doc_file.go Outdated Show resolved Hide resolved
internal/gengapic/doc_file.go Outdated Show resolved Hide resolved
codyoss and others added 2 commits February 23, 2022 10:47
Co-authored-by: Noah Dietz <noahdietz@users.noreply.github.com>
Co-authored-by: Noah Dietz <noahdietz@users.noreply.github.com>
@codyoss codyoss merged commit 7fe9fa5 into googleapis:main Feb 23, 2022
@codyoss codyoss deleted the version-refactor branch February 23, 2022 20:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants