Skip to content

protos: Cleanup unused imports#18024

Merged
mattklein123 merged 1 commit intoenvoyproxy:mainfrom
phlax:protos-cleanup-imports
Sep 9, 2021
Merged

protos: Cleanup unused imports#18024
mattklein123 merged 1 commit intoenvoyproxy:mainfrom
phlax:protos-cleanup-imports

Conversation

@phlax
Copy link
Copy Markdown
Member

@phlax phlax commented Sep 8, 2021

Signed-off-by: Ryan Northey ryan@synca.io

Commit Message: protos: Cleanup unused imports
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Deprecated:]
[Optional API Considerations:]

Signed-off-by: Ryan Northey <ryan@synca.io>
@repokitteh-read-only
Copy link
Copy Markdown

CC @envoyproxy/api-shepherds: Your approval is needed for changes made to api/envoy/.
envoyproxy/api-shepherds assignee is @adisuissa
CC @envoyproxy/api-watchers: FYI only for changes made to api/envoy/.

🐱

Caused by: #18024 was opened by phlax.

see: more, trace.

Copy link
Copy Markdown
Contributor

@adisuissa adisuissa left a comment

Choose a reason for hiding this comment

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

/lgtm api

@repokitteh-read-only repokitteh-read-only bot removed the api label Sep 9, 2021
@phlax
Copy link
Copy Markdown
Member Author

phlax commented Sep 9, 2021

@zuercher could i get a stamp for this one - its pretty straightforward cleanup

@mattklein123 mattklein123 merged commit f08b23d into envoyproxy:main Sep 9, 2021
@phlax
Copy link
Copy Markdown
Member Author

phlax commented Sep 9, 2021

wow 4s that was fast turnaround!

@zuercher
Copy link
Copy Markdown
Member

zuercher commented Sep 9, 2021

Everybody clicking on this at the same time. 😂

tyxia pushed a commit to tyxia/envoy that referenced this pull request Sep 21, 2021
Signed-off-by: Ryan Northey <ryan@synca.io>

Signed-off-by: phlax <phlax@users.noreply.github.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.

4 participants