search

Rdatatable / data.table

R's data.table package extends data.frame:
http://r-datatable.com
Mozilla Public License 2.0
3.53k stars 965 forks source link

Add clang-tidy to CI #2781

Open mattdowle opened 6 years ago

mattdowle commented 6 years ago

As suggested by @michaelquinn32 here: https://github.com/Rdatatable/data.table/pull/2738#issuecomment-380888101

jangorecki commented 6 months ago

https://clang.llvm.org/extra/clang-tidy/

This could be added on top of our current clang test job: https://github.com/Rdatatable/data.table/blob/685d51d3dd0deab81c1f6e4a392a89134307fbfb/.gitlab-ci.yml#L179-L183

MichaelChirico commented 6 months ago

https://github.com/Rdatatable/data.table/pull/4859

Is another PR fixing an issue identified by clang-tidy, as evidence that we could use CI to prevent backsliding on this type of thing.