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

Introduce language v1 proto and yaml file #190

Merged
merged 4 commits into from
Nov 14, 2016
Merged

Conversation

jmuk
Copy link
Contributor

@jmuk jmuk commented Nov 9, 2016

No description provided.

@jmuk
Copy link
Contributor Author

jmuk commented Nov 9, 2016

cc: @bjwatson @anthmgoogle
Note that this PR will not be merged until it's ready.

@jmuk jmuk changed the title Introduce language v1 proto. Introduce language v1 proto and yaml file Nov 9, 2016
@bjwatson
Copy link
Contributor

bjwatson commented Nov 9, 2016

@jmuk LGTM, once it's time to merge this.

@bjwatson
Copy link
Contributor

bjwatson commented Nov 9, 2016

FYI @swcloud, @jskeet, @jba, and @michaelbausor. These are the protos and config files you will need to generate gRPC and GAPIC code for NL v1.

@bjwatson bjwatson self-assigned this Nov 9, 2016
@jba
Copy link

jba commented Nov 9, 2016

@pongad

@pongad
Copy link
Contributor

pongad commented Nov 9, 2016

@jba Thank you for pinging me. I'll update genproto and cloud.google.com/go and keep you posted.

@bjwatson
Copy link
Contributor

@jmuk You have my LGTM from last week.

@jmuk
Copy link
Contributor Author

jmuk commented Nov 14, 2016

The new proto file has been merged through the automated tool. Now this PR is only for yaml files.

@jmuk
Copy link
Contributor Author

jmuk commented Nov 14, 2016

Note to language owners: if you already generated the GAPIC files last week with this PR, you don't have to regenerate them. The proto pushed to master is slightly different on some comments, but it won't affect GAPIC.

@jmuk jmuk merged commit 6a0a880 into googleapis:master Nov 14, 2016
@bjwatson
Copy link
Contributor

Thanks @jmuk!

@swcloud
Copy link
Contributor

swcloud commented Nov 15, 2016

@jmuk Good to know!

@jmuk jmuk deleted the language branch November 23, 2016 17:50
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.

5 participants