Update cpp-lint workflow configuration for ignored libs and embedded clang-tidy file #17
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.
Type
Goals
notstd
libs since we use a different coding style for those..clang-tidy
file checks with cpp-lint workflow.Technical Details
ignore
option withnotstd
directories.clang-tidy
option with empty value, enabling use of repo-embedded.clang-tidy
file.Test Results
None, this will be tested on the next PR.
Reviewer Focus
None
Future Work
The
.clang-tidy
file needs to be modified such that external dependencies such as CLI11 and Catch2 are ignored. Since these files are compiled and get included in thecompile_commands.json
file, it's unclear how to do this.Checklist
all
compiles cleanly.