envoyproxy / envoy

Cloud-native high-performance edge/middle/service proxy
https://www.envoyproxy.io
Apache License 2.0
24.67k stars 4.75k forks source link

Cleanup/address clang-tidy nolints #29846

Open phlax opened 11 months ago

phlax commented 11 months ago

As part of the drive to restore/fix clang-tidy testing to ci a lot of lines have NOLINTED

Some of these have ~trivial fixes

Some point to more serious (ie mem-related) issues and should probably be looked at further - a subset of these are non-testing code

phlax commented 11 months ago

cc @wbpcode

github-actions[bot] commented 10 months ago

This issue has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in the next 7 days unless it is tagged "help wanted" or "no stalebot" or other activity occurs. Thank you for your contributions.