Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add check to ensure that files in tensorflow/lite/micro have proper license and formatting #45674

Merged
merged 18 commits into from
Dec 15, 2020
Merged

Add check to ensure that files in tensorflow/lite/micro have proper license and formatting #45674

Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
7564497
Add check to ensure that files in tensorflow/lite/micro have proper l…
advaitjain Dec 14, 2020
7a60c64
remove extra .cmd exclusion.
advaitjain Dec 14, 2020
5f8e33f
Also add in presubmit check for clang-format.
advaitjain Dec 15, 2020
170afea
Also add in presubmit check for clang-format.
advaitjain Dec 15, 2020
fcfcdf1
add another exclusion.
advaitjain Dec 15, 2020
5b20862
Merge remote-tracking branch 'upstream/master' into license-check
advaitjain Dec 15, 2020
bc909c7
Merge branch 'license-check' of github.com:advaitjain/tensorflow into…
advaitjain Dec 15, 2020
b65ee70
install clang-format on the TFLM docker image.
advaitjain Dec 15, 2020
914da4d
workaround to use pigweed when we do not have a git repository.
advaitjain Dec 15, 2020
40813ad
small fix
advaitjain Dec 15, 2020
b4efe0d
fix typo
advaitjain Dec 15, 2020
9098df4
some more fixes for the local git repo for CI.
advaitjain Dec 15, 2020
d590360
fix remaining issues and tested locally with the PRESUBMIT option.
advaitjain Dec 15, 2020
998e3c9
Merge remote-tracking branch 'upstream/master' into license-check
advaitjain Dec 15, 2020
646b965
Manually ignore some sections of the code.
advaitjain Dec 15, 2020
0e30134
Merge remote-tracking branch 'upstream/master' into license-check
advaitjain Dec 15, 2020
491e13a
use more recent clang-format on the docker image.
advaitjain Dec 15, 2020
8f623c2
get latest and greatest clang-format.
advaitjain Dec 15, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
30 changes: 30 additions & 0 deletions tensorflow/lite/micro/tools/ci_build/test_all.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,36 @@ if [ -d tensorflow/lite/micro/tools/make/downloads ]; then
exit 1
fi

# explicitly call third_party_downloads since we need pigweed for the license
# and clang-format checks.
make -f tensorflow/lite/micro/tools/make/Makefile third_party_downloads

# Check for license with the necessary exclusions.
tensorflow/lite/micro/tools/make/downloads/pigweed/pw_presubmit/py/pw_presubmit/pigweed_presubmit.py \
tensorflow/lite/micro \
-p copyright_notice \
-e micro/tools/make/targets/ecm3531 \
-e BUILD\
-e leon_commands \
-e Makefile \
-e "\.bzl" \
-e "\.cmd" \
-e "\.cmd" \
-e "\.conf" \
-e "\.defaults" \
-e "\.h5" \
-e "\.ipynb" \
-e "\.inc" \
-e "\.lcf" \
-e "\.ld" \
-e "\.lds" \
-e "\.projbuild" \
-e "\.properties" \
-e "\.resc" \
-e "\.robot" \
-e "\.txt" \
-e "\.tpl" \
--output-directory /tmp

# We are moving away from having the downloads and installations be part of the
# Makefile. As a result, we need to manually add the downloads in this script.
Expand Down