Update to the latest buildifier release. (#633)

* Update to the latest buildifier release.

For the previous buildifier version, the exit code would always be 0
(regardless of whether files needed formatting or not).

Manually confirmed that with version 4.2.3 the test_code_style.sh script
correctly errors out when a BUILD file has formatting errors (and that
it doesn't with the previous version).

Also,
 * Updated the docker container and pointing to the new container.
 * Fix the formatting errors in .bzl file that crept in because this check was not working in OSS.

BUG=http://b/203803273

* Exclude the downloads directory.

* Updated docs based on review comments.
This commit is contained in:
Advait Jain 2021-10-27 15:46:49 -07:00 committed by GitHub
parent 0beb99eb2a
commit 6cb1580846
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
6 changed files with 22 additions and 17 deletions

View file

@ -1,8 +1,9 @@
<!--ts-->
* [Continuous Integration System](#continuous-integration-system)
* [Automated Sync from the Tensorflow Repository](#automated-sync-from-the-tensorflow-repository)
* [Third Party GitHub Actions Used](#third-party-github-actions-used)
* [Third Party GitHub Actions](#third-party-github-actions)
<!-- Added by: advaitjain, at: Thu 29 Apr 2021 12:54:23 PM PDT -->
<!-- Added by: advaitjain, at: Wed Oct 27 03:31:51 PM PDT 2021 -->
<!--te-->
@ -12,9 +13,9 @@
* See the [github workflow files](.github/workflows/ci.yml) for details on
exactly what is run as part of the GitHub Actions CI.
* Tests can also be run from within a docker container:
* Tests can also be run from within a docker container, which can be built with:
```
docker build -t tflm-test -f ci/Dockerfile.micro .
docker build -t tflm-ci -f ci/Dockerfile.micro .
```
or use the tflm-ci docker image from [here](https://github.com/users/TFLM-bot/packages/container/package/tflm-ci).