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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: update all protos and pb2 files #92

Merged
merged 4 commits into from
Mar 17, 2022

Conversation

parthea
Copy link
Contributor

@parthea parthea commented Mar 17, 2022

Fixes #83 馃

BEGIN_COMMIT_OVERRIDE
feat: update all protos and pb2 files

feat: add google/api/error_reason.proto
feat: add google/api/visibility.proto
feat: add google/type/decimal.proto
feat: add google/type/interval.proto
feat: add google/type/localized_text.proto
feat: add google/type/phone_number.proto
fix: remove deprecated fields aliases and features from google/api/endpoint.proto
END_COMMIT_OVERRIDE

@parthea parthea requested a review from a team as a code owner March 17, 2022 10:24
@parthea parthea added the owlbot:ignore instruct owl-bot to ignore a PR label Mar 17, 2022
@parthea parthea marked this pull request as draft March 17, 2022 10:25
@parthea parthea force-pushed the use-protos-from-googleapis-directly branch from ea8384d to dd2ffab Compare March 17, 2022 10:33
@parthea parthea added owlbot:run Add this label to trigger the Owlbot post processor. and removed owlbot:ignore instruct owl-bot to ignore a PR labels Mar 17, 2022
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Mar 17, 2022
@parthea parthea added kokoro:run Add this label to force Kokoro to re-run the tests. kokoro:force-run Add this label to force Kokoro to re-run the tests. owlbot:run Add this label to trigger the Owlbot post processor. labels Mar 17, 2022
@yoshi-kokoro yoshi-kokoro removed kokoro:run Add this label to force Kokoro to re-run the tests. kokoro:force-run Add this label to force Kokoro to re-run the tests. labels Mar 17, 2022
@parthea parthea force-pushed the use-protos-from-googleapis-directly branch 2 times, most recently from 46ba7a5 to 18b37a9 Compare March 17, 2022 10:43
@parthea parthea marked this pull request as ready for review March 17, 2022 10:46
@parthea parthea requested a review from busunkim96 March 17, 2022 10:47
@parthea parthea force-pushed the use-protos-from-googleapis-directly branch from 968e7fc to af806be Compare March 17, 2022 18:16
@parthea parthea force-pushed the use-protos-from-googleapis-directly branch from af806be to 369a626 Compare March 17, 2022 18:18
@parthea parthea force-pushed the use-protos-from-googleapis-directly branch from 70592f7 to 4a3671f Compare March 17, 2022 18:20
Comment on lines -55 to -56
// The list of features enabled on this endpoint.
repeated string features = 4;
Copy link
Contributor

Choose a reason for hiding this comment

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

This looks like a breaking change.

Copy link
Contributor Author

@parthea parthea Mar 17, 2022

Choose a reason for hiding this comment

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

Great catch! I'll update the release notes. This breaking change is from googleapis/googleapis@ca1372c. There is a comment that says features field is no longer used. It has been removed from the API runtime for awhile. Based on the fact that's it's already removed from the API runtime, the breaking change already happened on the backend and existing code would already fail. In this case, we won't bump the major version.

@parthea parthea merged commit 62c04b8 into main Mar 17, 2022
@parthea parthea deleted the use-protos-from-googleapis-directly branch March 17, 2022 20:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
owlbot:run Add this label to trigger the Owlbot post processor.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use googleapis/googleapis as the source for protos instead of googleapis/api-common-protos
3 participants