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

tools: API boosting support for renames. #9442

Merged
merged 6 commits into from
Dec 26, 2019

Conversation

htuch
Copy link
Member

@htuch htuch commented Dec 20, 2019

The type database now has rename information.

Also some misc. fixes for enum and generated static method handling
support.

Part of #8082

Signed-off-by: Harvey Tuch htuch@google.com

The type database now has rename information.

Also some misc. fixes for enum and generated static method handling
support.

Signed-off-by: Harvey Tuch <htuch@google.com>
@repokitteh-read-only
Copy link

CC @envoyproxy/api-shepherds: Your approval is needed for changes made to api/.

🐱

Caused by: #9442 was synchronize by htuch.

see: more, trace.

Signed-off-by: Harvey Tuch <htuch@google.com>
Signed-off-by: Harvey Tuch <htuch@google.com>
Copy link
Member

@lizan lizan left a comment

Choose a reason for hiding this comment

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

Fine leave a TODO to unblock your upcoming PRs.

Signed-off-by: Harvey Tuch <htuch@google.com>
@htuch htuch merged commit 2c8992b into envoyproxy:master Dec 26, 2019
sthagen added a commit to sthagen/envoyproxy-envoy that referenced this pull request Dec 26, 2019
tools: API boosting support for renames. (envoyproxy#9442)
prakhag1 pushed a commit to prakhag1/envoy that referenced this pull request Jan 3, 2020
The type database now has rename information.

Also some misc. fixes for enum and generated static method handling
support.

Part of envoyproxy#8082

Signed-off-by: Harvey Tuch <htuch@google.com>
Signed-off-by: Prakhar <prakhar_au@yahoo.com>
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.

3 participants