Skip to content

Commit 05320e4

Browse files
committed
Fix review comments
1 parent 6123b85 commit 05320e4

File tree

2 files changed

+1
-3
lines changed

2 files changed

+1
-3
lines changed

.github/workflows/tools-rippled.yml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -98,7 +98,6 @@ jobs:
9898
DOXYGEN_VERSION=${{ env.DOXYGEN_VERSION }}
9999
GCC_VERSION=${{ env.GCC_VERSION }}
100100
GRAPHVIZ_VERSION=${{ env.GRAPHVIZ_VERSION }}
101-
LLVM_VERSION=${{ env.LLVM_VERSION }}
102101
PRE_COMMIT_VERSION=${{ env.PRE_COMMIT_VERSION }}
103102
context: docker/tools-rippled
104103
outputs: type=image,name=${{ env.CONTAINER_IMAGE }},push-by-digest=true,name-canonical=true,push=${{ env.PUSH }}
@@ -128,7 +127,6 @@ jobs:
128127
strategy:
129128
matrix:
130129
tool:
131-
- clang-tidy
132130
- documentation
133131
- pre-commit
134132
- signing

docker/ubuntu/Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -228,7 +228,7 @@ printf "%s\n" \
228228
"Acquire::AllowWeakRepositories \"true\";" \
229229
| tee /etc/apt/apt.conf.d/99llvm-allow-weak
230230

231-
# Add LLVM repository for versions above 20
231+
# Add LLVM repository for versions above 20 because it is not available in the default repo
232232
if [[ "${CLANG_VERSION}" -gt 20 ]]; then
233233
apt-get update
234234
apt-get install -y --no-install-recommends lsb-release software-properties-common gnupg

0 commit comments

Comments
 (0)