envoyproxy / envoyproxy/envoy

Add "include-what-you-use" to CI

Open
#26,797 13 comments 0 reactions 0 assignees View on GitHub
area/ci enhancement no stalebot
Dominant language
C++
Stars
28.9k
Forks
5.6k
Avg merge
1d 20h
Merged PRs (30d)
437

Description

Currently our builds can take an age with the smallest of change.

Probably we need to do some tree shaking and improve inputs/dependencies between build targets - im guessing esp in the test code

It seems we dont currently check for used includes (not 100% but that seems to be the case)

Perhaps we can add https://github.com/include-what-you-use/include-what-you-use at least to get the signal if not to fix immediately

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.