Releases: ZedThree/clang-tidy-review
Fix for path replacement
Fix issue with replacing paths: #21 (thanks to @kgfoundrydig)
Enable clang-tidy-11
Adds clang-tidy-11
as an option. Thanks to @ghutchis
Minor fix: handle "no newline at end of file"
Diffs can contain "No newline at end of file" which needs special handling when constructing the lookup table
Two minor bug fixes
- Don't try to create comments on files not in the PR
- Exit early if the PR only removes lines
Avoid apt warning when building
Uses apt-get
instead of apt
to install packages to avoid warning about apt
not having a stable CLI
Fixes for passing include/exclude arguments
Previously it was tricking to pass through quoted arguments for the include/exclude options as they passed through GitHub Actions, then Docker, then bash, then finally Python. They should now work properly
Limit maximum comments posted at one time
It seems the GitHub API might only accept a limited number of comments at once, so clang-tidy-review
will only attempt to post the first max_comments
of them (default 25, as this has worked for me).
Fixes for larger projects
- Replace absolute paths in
compile_commands.json
if it exists- Needed due to container-based Actions being mounted in a different location, and
clang-tidy
not accepting relative paths
- Needed due to container-based Actions being mounted in a different location, and
- Add argument to install additional apt packages
- Needed to bring in headers for third-party dependencies
- Fix for PRs with multiple files
- Fix for whitespace in code-blocks in comments
Replace curl with requests
v0.2.0 Update README and add license info
Initial release - v0.1.0
Initial release, may be some things to iron out still