Skip to content

Commit adebdf0

Browse files
authored
[CI] Add help for IWYU and clang-tidy. (#4698)
1 parent e8a9ba9 commit adebdf0

2 files changed

Lines changed: 14 additions & 0 deletions

File tree

‎.github/workflows/clang-tidy.yaml‎

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -108,6 +108,13 @@ jobs:
108108
readonly COUNT="$WARNING_COUNT"
109109
readonly LIMIT="$WARNING_LIMIT"
110110
111+
# Print instructions to help investigations
112+
echo "::notice::See instructions below to investigate warnings"
113+
echo "1) Find the previous job in CI, named 'Upload warning report'"
114+
echo "2) In the job logs, search for 'Artifact download URL:'"
115+
echo "3) Upload the report given by the URL"
116+
echo "4) Read the warnings in the report, and adjust the code accordingly."
117+
111118
echo "clang-tidy reported ${COUNT} unique warning(s) with preset '${CMAKE_OPTIONS}'"
112119
echo "Limit is ${LIMIT}"
113120

‎.github/workflows/iwyu.yml‎

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -85,6 +85,13 @@ jobs:
8585
echo "include-what-you-use reported ${WARNING_COUNT} warning(s) with cmake options preset '${{ matrix.cmake_options }}'"
8686
# Acceptable limit, to decrease over time down to 0
8787
readonly WARNING_LIMIT=${{ matrix.warning_limit }}
88+
89+
# Print instructions to help investigations
90+
echo "::notice::See instructions below to investigate warnings"
91+
echo "1) Find the previous job in CI, named 'Build with include-what-you-use'"
92+
echo "2) In the job logs, search for 'Warning: include-what-you-use reported diagnostics:'"
93+
echo "3) Read the warning messages, and adjust the code accordingly."
94+
8895
# FAIL the build if WARNING_COUNT > WARNING_LIMIT
8996
if [ $WARNING_COUNT -gt $WARNING_LIMIT ] ; then
9097
echo "include-what-you-use reported ${WARNING_COUNT} warning(s) exceeding the existing warning limit of ${WARNING_LIMIT} with cmake options preset '${{ matrix.cmake_options }}'"

0 commit comments

Comments
 (0)