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!: add owlbot automated updates #54

Merged
merged 19 commits into from
Apr 28, 2023
Merged

feat!: add owlbot automated updates #54

merged 19 commits into from
Apr 28, 2023

Conversation

bshaffer
Copy link
Contributor

@bshaffer bshaffer commented Apr 4, 2023

Adds compliance with AIP-213

Note: I've removed all the alias files because we are going to tag a new major version with the new protobuf updates. And without deleting these in the PR, the owlbot update was causing the tests to fail.

see #55 for what this repo looks like after removing everything in src and metadata and regenerating with owlbot.

@bshaffer bshaffer changed the title chore: add owlbot chore: add owlbot automated updates Apr 10, 2023
@bshaffer bshaffer marked this pull request as ready for review April 10, 2023 02:49
@bshaffer bshaffer requested a review from a team as a code owner April 10, 2023 02:49
@bshaffer bshaffer changed the title chore: add owlbot automated updates feat!: add owlbot automated updates Apr 25, 2023
@bshaffer bshaffer requested a review from noahdietz April 28, 2023 18:10
@bshaffer bshaffer merged commit 6d9134d into main Apr 28, 2023
@bshaffer bshaffer deleted the add-owlbot branch April 28, 2023 18:55
("iamlogging", "iam"),
("rpc", "rpc"),
("type", "type"),
]
Copy link
Contributor Author

Choose a reason for hiding this comment

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

FYI @chingor13 @SurferJeffAtGoogle I am doing some unusual things with version and the copy paths here. I am not sure how else to do this, but if there's a better way please let me know!

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.

Automate refresh of common protos
2 participants