valitydev/osquery-1
14
0
Fork 0
mirror of https://github.com/valitydev/osquery-1.git synced 2024-11-07 01:55:20 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Include header files in the test of format-check.py (#6157)

Browse Source 
If we don't include them and if the commit we're on requires formatting
on a header file, the test will fail.
We don't want it to fail because it's not the job of this test
to verify that the commit is properly formatted.
Moreover, it just makes sense to test that the formatting script
can properly reach all files it has to.
This commit is contained in:
Stefano Bonicatti 2020-01-04 01:57:27 +01:00 committed by Teddy Reed
parent 80ed884273
commit 36a66ff6ab
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
tools/formatting/format-test.sh
View File

@ -19,8 +19,8 @@ build_dir=${args[1]}
set -e set -e
# List all .cpp files to format, skipping all the ones inside the libraries folder, sort them and save the list on a file # List all files to format, skipping all the ones inside the libraries folder, sort them and save the list on a file
find . -type d -path ./libraries -prune -o -type f -name "*.cpp" -print | sort > $build_dir/files_to_format.txt find . -type d -path ./libraries -prune -o -type f -regex ".*\.cpp$\|.*\.h$\|.*\.hpp$" -print | sort > $build_dir/files_to_format.txt
# Read back the list of files, modify them appending two newlines, so that the format_check.py is triggered # Read back the list of files, modify them appending two newlines, so that the format_check.py is triggered
while read line; do echo -e "\n\n" >> $line; done <<< `cat $build_dir/files_to_format.txt` while read line; do echo -e "\n\n" >> $line; done <<< `cat $build_dir/files_to_format.txt`