Skip to content

Commit b480fc0

Browse files
committed
ci: relax include cleaner lint
1 parent 2cd6e8c commit b480fc0

3 files changed

Lines changed: 3 additions & 6 deletions

File tree

.clang-tidy

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,6 @@ Checks: |
2121
clang-analyzer-*,
2222
google-*,
2323
modernize-*,
24-
misc-include-cleaner,
2524
readability-identifier-naming,
2625
readability-isolate-declaration,
2726
-modernize-use-nodiscard,
@@ -42,7 +41,5 @@ CheckOptions:
4241
value: '_'
4342
- key: modernize-use-scoped-lock.WarnOnSingleLocks
4443
value: 'false'
45-
- key: misc-include-cleaner.IgnoreHeaders
46-
value: 'arrow/.*;avro/.*;aws/.*;cpr/.*;gmock/.*;gtest/.*;nanoarrow/.*;nlohmann/.*;parquet/.*;roaring/.*;spdlog/.*;sqlpp23/.*;thrift/.*'
4744

4845
HeaderFilterRegex: 'src/iceberg|example'

.github/workflows/aws_test.yml

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -92,7 +92,7 @@ jobs:
9292
path: /usr/local/share/vcpkg/installed
9393
key: vcpkg-x64-linux-aws-sdk-cpp-s3-${{ matrix.s3 }}-sigv4-${{ matrix.sigv4 }}-${{ hashFiles('.github/workflows/aws_test.yml') }}
9494
- name: Install AWS SDK via vcpkg
95-
if: ${{ startsWith(matrix.runs-on, 'ubuntu') && matrix.bundle_awssdk == 'OFF' && steps.vcpkg-cache.outputs.cache-hit != 'true' }}
95+
if: ${{ startsWith(matrix.runs-on, 'ubuntu') && matrix.bundle_awssdk == 'OFF' }}
9696
shell: bash
9797
# Retry to ride out transient GitHub/mirror download failures (504s).
9898
run: |
@@ -169,7 +169,6 @@ jobs:
169169
path: /usr/local/share/vcpkg/installed
170170
key: vcpkg-x64-linux-aws-sdk-cpp-core-${{ hashFiles('.github/workflows/aws_test.yml') }}
171171
- name: Install AWS SDK via vcpkg
172-
if: ${{ steps.vcpkg-cache.outputs.cache-hit != 'true' }}
173172
shell: bash
174173
# Retry to ride out transient GitHub/mirror download failures (504s).
175174
run: |

.github/workflows/cpp-linter.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,7 @@ jobs:
4444
runs-on: ubuntu-26.04
4545
permissions:
4646
contents: read
47+
issues: write
4748
pull-requests: write
4849
env:
4950
SCCACHE_DIR: ${{ github.workspace }}/.sccache
@@ -103,7 +104,7 @@ jobs:
103104
version: 22
104105
files-changed-only: true
105106
lines-changed-only: true
106-
thread-comments: true
107+
thread-comments: ${{ github.event.pull_request.head.repo.full_name == github.repository }}
107108
ignore: 'build|cmake_modules|ci|src/iceberg/catalog/hive/gen-cpp'
108109
database: build
109110
verbosity: 'debug'

0 commit comments

Comments
 (0)