Skip to content

Commit

Permalink
Merge pull request #121 from daljit46/clang_tidy_18
Browse files Browse the repository at this point in the history
Add clang-tidy 18
  • Loading branch information
ZedThree authored May 13, 2024
2 parents 3ff13b4 + 9bde8c5 commit b719737
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 7 deletions.
5 changes: 3 additions & 2 deletions Dockerfile
Original file line number Diff line number Diff line change
@@ -1,14 +1,15 @@
FROM ubuntu:23.04
FROM ubuntu:24.04

RUN apt update && \
DEBIAN_FRONTEND=noninteractive \
apt-get install -y --no-install-recommends\
build-essential cmake git \
tzdata \
clang-tidy-13 \
clang-tidy-14 \
clang-tidy-15 \
clang-tidy-16 \
clang-tidy-17 \
clang-tidy-18 \
python3 \
python3-pip \
&& rm -rf /var/lib/apt/lists/
Expand Down
5 changes: 2 additions & 3 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -88,9 +88,8 @@ at once, so `clang-tidy-review` will only attempt to post the first
- `base_dir`: Absolute path to initial working directory
`GITHUB_WORKSPACE`.
- default: `GITHUB_WORKSPACE`
- `clang_tidy_version`: Version of clang-tidy to use; one of
13, 14, 15, 16
- default: '16'
- `clang_tidy_version`: Version of clang-tidy to use; one of 14, 15, 16, 17, 18
- default: '18'
- `clang_tidy_checks`: List of checks
- default: `'-*,performance-*,readability-*,bugprone-*,clang-analyzer-*,cppcoreguidelines-*,mpi-*,misc-*'`
- `config_file`: Path to clang-tidy config file, replaces `clang_tidy_checks`
Expand Down
4 changes: 2 additions & 2 deletions action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,8 @@ inputs:
default: ${{ github.workspace }}
require: false
clang_tidy_version:
description: 'Version of clang-tidy to use; one of 13, 14, 15, 16'
default: '16'
description: 'Version of clang-tidy to use; one of 14, 15, 16, 17, 18'
default: '18'
required: false
clang_tidy_checks:
description: 'List of checks'
Expand Down

0 comments on commit b719737

Please sign in to comment.