A Github Action for linting C/C++ code integrating clang-tidy and clang-format to collect feedback provided in the form of thread comments and/or annotations.
v2
- Change action from using docker to composite steps
- improve workflow runs times from 1m 24s (currently) to 6-20s.
- better support for the database input option (which is currently broken with the docker env).
- better support cross-compilation
- better support 3rd party libraries
- Includes many issues and enhancements. See #87 for details.
Refer here for previous versions.
Create a new GitHub Actions workflow in your project, e.g. at .github/workflows/cpp-linter.yml
The content of the file should be in the following format.
name: cpp-linter
on: pull_request
jobs:
cpp-linter:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- uses: cpp-linter/cpp-linter-action@v2
id: linter
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
with:
style: file
- name: Fail fast?!
if: steps.linter.outputs.checks-failed > 0
run: echo "Some files failed the linting checks!"
# for actual deployment
# run: exit 1
- Description: The style rules to use.
- Set this to 'file' to have clang-format use the closest relative .clang-format file.
- Set this to a blank string (
''
) to disable the use of clang-format entirely. - Any code style supported by the specified version of clang-format.
- Default: 'llvm'
- Description: The file extensions to run the action against. This is a comma-separated string.
- Default: 'c,h,C,H,cpp,hpp,cc,hh,c++,h++,cxx,hxx'
- Description: Comma-separated list of globs with optional
-
prefix. Globs are processed in order of appearance in the list. Globs without-
prefix add checks with matching names to the set, globs with the-
prefix remove checks with matching names from the set of enabled checks. This option's value is appended to the value of the 'Checks' option in a .clang-tidy file (if any).- It is possible to disable clang-tidy entirely by setting this option to
'-*'
. - It is also possible to rely solely on a .clang-tidy config file by specifying this option as a blank string (
''
).
- It is possible to disable clang-tidy entirely by setting this option to
- Default: 'boost-*,bugprone-*,performance-*,readability-*,portability-*,modernize-*,clang-analyzer-*,cppcoreguidelines-*'
- Description: The relative path to the repository root directory. This path is relative to the path designated as the runner's GITHUB_WORKSPACE environment variable.
- Default: '.'
- Description: The desired version of the clang-tools to use. Accepted options are strings which can be 15, 14, 13, 12, 11, 10, 9, 8,7, 6, 5, 4 or 3.9.
- Set this option to a blank string (
''
) to use the platform's default installed version. - This value can also be a path to where the clang tools are installed (if using a custom install location).
- Set this option to a blank string (
- Default: '12'
- Description: This controls the action's verbosity in the workflow's logs. Supported options are defined by the python logging library's log levels. This option does not affect the verbosity of resulting thread comments or file annotations.
- Default: '10'
- Description: This controls what part of the files are analyzed. The following values are accepted:
- false: All lines in a file are analyzed.
- true: Only lines in the diff that contain additions are analyzed.
- diff: All lines in the diff are analyzed (including unchanged lines but not subtractions).
- Default: false.
- Description: Set this option to false to analyze any source files in the repo. This is automatically enabled if lines-changed-only is enabled.
- Default: true
- NOTE: The
GITHUB_TOKEN
should be supplied when running on a private repository with this option enabled, otherwise the runner does not not have the privilege to list changed files for an event. See Authenticating with the GITHUB_TOKEN
- Description: Set this option with string of path(s) to ignore.
- In the case of multiple paths, you can use a pipe character (
|
) to separate the multiple paths. Multiple lines are forbidden as an input to this option; it must be a single string. - This can also have files, but the file's relative path has to be specified as well.
- There is no need to use
./
for each entry; a blank string (''
) represents the repo-root path (specified by therepo-root
input option). - Submodules are automatically ignored. Hidden directories (beginning with a
.
) are also ignored automatically. - Prefix a path with a bang (
!
) to make it explicitly not ignored. The order of multiple paths does not take precedence. The!
prefix can be applied to a submodule's path (if desired) but not hidden directories. - Glob patterns are not supported here. All asterisk characters (
*
) are literal.
- In the case of multiple paths, you can use a pipe character (
- Default: '.github'
- Description: Set this option to false to disable the use of thread comments as feedback.
- To use thread comments, the
GITHUB_TOKEN
(provided by Github to each repository) must be declared as an environment variable. See Authenticating with the GITHUB_TOKEN
- To use thread comments, the
- Default: false
- NOTE: If run on a private repository, then this feature is disabled because the GitHub REST API behaves differently for thread comments on a private repository.
- Description: Set this option to false to disable the use of file annotations as feedback.
- Default: true
- Description: The directory containing compilation database (like compile_commands.json) file.
- Default: ''
- Description: A string of extra arguments passed to clang-tidy for use as compiler arguments (like
-std=c++14 -Wall
). - Default: ''
This action creates 1 output variable named checks-failed
. Even if the linting checks fail for source files this action will still pass, but users' CI workflows can use this action's output to exit the workflow early if that is desired.
You can show C/C++ Linter Action status with a badge in your repository README
Example
[![cpp-linter](https://github.com/cpp-linter/cpp-linter-action/actions/workflows/cpp-linter.yml/badge.svg)](https://github.com/cpp-linter/cpp-linter-action/actions/workflows/cpp-linter.yml)
To provide feedback (requesting a feature or reporting a bug) please post to issues.
The scripts and documentation in this project are released under the MIT License