Add lint to CI #15
Comments
|
Is there any preset github action for this that we can refer to? |
westonpace
pushed a commit
that referenced
this issue
Mar 22, 2023
This PR should be the first PR of issue #15, and then the second [PR](#30) will use this for lint check. - please see https://github.com/apache/skywalking-eyes for license check - run-clang-tidy.py : a python script used to check cpp project with clang-tidy Co-authored-by: chaojun-zhang <zcj23085@gmail.com>
EpsilonPrime
pushed a commit
to EpsilonPrime/substrait-cpp
that referenced
this issue
Mar 24, 2023
This PR should be the first PR of issue substrait-io#15, and then the second [PR](substrait-io#30) will use this for lint check. - please see https://github.com/apache/skywalking-eyes for license check - run-clang-tidy.py : a python script used to check cpp project with clang-tidy Co-authored-by: chaojun-zhang <zcj23085@gmail.com>
EpsilonPrime
pushed a commit
to EpsilonPrime/substrait-cpp
that referenced
this issue
May 6, 2023
This PR should be the first PR of issue substrait-io#15, and then the second [PR](substrait-io#30) will use this for lint check. - please see https://github.com/apache/skywalking-eyes for license check - run-clang-tidy.py : a python script used to check cpp project with clang-tidy Co-authored-by: chaojun-zhang <zcj23085@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Required:
cmake_format
clang-format
Some kind of spdx licensing header check
Consider:
clang-tidy
iwyu
The text was updated successfully, but these errors were encountered: