search

apache / arrow-adbc

Database connectivity API standard and libraries for Apache Arrow
https://arrow.apache.org/adbc/
Apache License 2.0
384 stars 97 forks source link

c: enable IWYU #814

Open lidavidm opened 1 year ago

lidavidm commented 1 year ago

We should ideally jump on this early or else it becomes kind of impossible to keep up (as already seen with clang-tidy)

WillAyd commented 1 year ago

I think CMake 3.27 will natively support IWYU and can report back useful error codes

re: https://gitlab.kitware.com/cmake/cmake/-/merge_requests/8406 and https://github.com/include-what-you-use/include-what-you-use/issues/985

lidavidm commented 1 year ago

Yes, I was trying to test it out but under Conda at least it seems to need more fiddling with include paths.