Add some additional tools to the CI #3275
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
P-E-P
force-pushed
the
clang-tidy
branch
7 times, most recently
from
8522432 to
4e3793d
Compare
P-E-P
force-pushed
the
clang-tidy
branch
3 times, most recently
from
5e163b4 to
2a1b269
Compare
dkm
reviewed
Nov 29, 2024
| jobs: | ||
| compilation-database: | ||
| env: | ||
| # Force locale, in particular for reproducible results re '.github/log_expected_warnings' (see below). |
| include-what-you-use: | ||
| needs: compilation-database | ||
| env: | ||
| # Force locale, in particular for reproducible results re '.github/log_expected_warnings' (see below). |
|
The build probably has to be parallelized in order to complete in time |
Bear doesn't like parallel jobs, IIRC there is a race condition on the jobserver, compiling from scratch with only one job shouldn't take that much time, at most 1h. Here we hit another bug in bear, which makes bear hang indefinitely even if the compilation was done. I'll probably get rid of bear anyway, I thought it would be great to have compilation database artifacts around but they are not as useful as I initially thought and I do not want to fight bear indefinitely. |
ChangeLog: * .github/workflows/lint.yml: New file. * .github/clang-tidy.yml: New file. Signed-off-by: Pierre-Emmanuel Patry <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Add clang-tidy and iwyu