apache / apache/arrow

[CI][C++] Add clang-tidy and IWYU C++ CI jobs

Open
#35,214 1 comment 1 reaction 1 assignee Claimed by @wjones127 View on GitHub
Component: C++ Component: Continuous Integration Type: enhancement
Dominant language
C++
Stars
17.1k
Forks
4.3k
Avg merge
3d 18h
Merged PRs (30d)
91

Description

### Describe the enhancement requested

AFAIK these two checks aren't run, but would be very valuable. However, both are slow, so it's most useful if we can run them only on files that have been changed. There is a script to detect changed files, but it is outdated:

https://github.com/apache/arrow/blob/ccc9d2c8da9220f9ed414ab8c71ed4af92fd5f34/cpp/build-support/get-upstream-commit.sh#L25

See also: https://github.com/apache/arrow/issues/16763

### Component(s)

C++, Continuous Integration

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.