Update by review

pull/24193/head
Esun Kim 4 years ago
parent 6d6648c7e1
commit 8db60763ca
  1. 2
      tools/distrib/clang_tidy_code.sh
  2. 4
      tools/run_tests/sanity/sanity_tests.yaml

@ -25,7 +25,7 @@ REPO_ROOT=$(pwd)
# let's get a list of them using query command and pass it to gen_compilation_database.py # let's get a list of them using query command and pass it to gen_compilation_database.py
export MANUAL_TARGETS=$(bazel query 'attr("tags", "manual", tests(//test/cpp/...))' | grep -v _on_ios) export MANUAL_TARGETS=$(bazel query 'attr("tags", "manual", tests(//test/cpp/...))' | grep -v _on_ios)
# generate a clang compilation database # generate a clang compilation database for all C/C++ sources in the repo.
tools/distrib/gen_compilation_database.py \ tools/distrib/gen_compilation_database.py \
--include_headers \ --include_headers \
--ignore_system_headers \ --ignore_system_headers \

@ -24,8 +24,8 @@
- script: tools/distrib/check_pytype.sh - script: tools/distrib/check_pytype.sh
- script: tools/distrib/clang_format_code.sh - script: tools/distrib/clang_format_code.sh
- script: tools/distrib/clang_tidy_code.sh - script: tools/distrib/clang_tidy_code.sh
# ClangTidy needs to run exclusively because it runs bazel making changes # ClangTidy needs to run exclusively because it uses files under the bazel output
# and it needs many CPUs to parse all source files to be diagnosed. # directory and this will be removed by another bazel invocation.
cpu_cost: 1000 cpu_cost: 1000
- script: tools/distrib/pylint_code.sh - script: tools/distrib/pylint_code.sh
- script: tools/distrib/python/check_grpcio_tools.py - script: tools/distrib/python/check_grpcio_tools.py

Loading…
Cancel
Save