Skip to content

cleanup: clang tidy naming#12181

Merged
alyssawilk merged 2 commits intoenvoyproxy:masterfrom
alyssawilk:naming
Jul 20, 2020
Merged

cleanup: clang tidy naming#12181
alyssawilk merged 2 commits intoenvoyproxy:masterfrom
alyssawilk:naming

Conversation

@alyssawilk
Copy link
Copy Markdown
Contributor

got bit by clang tidy "rename this thing you didn't write" one too many time...

Risk Level: low (function renames)
Testing: n/a
Docs Changes: n/a
Release Notes: n/a

Signed-off-by: Alyssa Wilk <alyssar@chromium.org>
jmarantz
jmarantz previously approved these changes Jul 20, 2020
Signed-off-by: Alyssa Wilk <alyssar@chromium.org>
Copy link
Copy Markdown
Member

@zuercher zuercher left a comment

Choose a reason for hiding this comment

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

Thank you!

@alyssawilk alyssawilk merged commit cd15dc4 into envoyproxy:master Jul 20, 2020
KBaichoo pushed a commit to KBaichoo/envoy that referenced this pull request Jul 30, 2020
Risk Level: low (function renames)
Testing: n/a
Docs Changes: n/a
Release Notes: n/a

Signed-off-by: Alyssa Wilk <alyssar@chromium.org>
Signed-off-by: Kevin Baichoo <kbaichoo@google.com>
scheler pushed a commit to scheler/envoy that referenced this pull request Aug 4, 2020
Risk Level: low (function renames)
Testing: n/a
Docs Changes: n/a
Release Notes: n/a

Signed-off-by: Alyssa Wilk <alyssar@chromium.org>
Signed-off-by: scheler <santosh.cheler@appdynamics.com>
@alyssawilk alyssawilk deleted the naming branch December 10, 2020 19:15
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