Skip to content

Commit

Permalink
Simplify clang-tidy; read directly from .clang-tidy config file.
Browse files Browse the repository at this point in the history
Older clang-tidy did not have --config-file yet, these days they
have, which simplifies things.

Fix issues found in editscript* which is now included in clang-tidy
runs.
  • Loading branch information
hzeller committed May 4, 2024
1 parent ec69cae commit b7ccea6
Show file tree
Hide file tree
Showing 4 changed files with 277 additions and 171 deletions.
3 changes: 2 additions & 1 deletion .github/bin/check-potential-problems.sh
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,8 @@ EXIT_CODE=0
#
# So, until these assumptions are fixed, we need to use absl::string_view that
# comes with the same implementation everywhere.
find . -name "*.h" -o -name "*.cc" | xargs grep -n "std::string_view"
find common verilog -name "*.h" -o -name "*.cc" | \
xargs grep -n "std::string_view"
if [ $? -eq 0 ]; then
echo "::error:: use absl::string_view instead of std::string_view"
echo
Expand Down
Loading

0 comments on commit b7ccea6

Please sign in to comment.