search

tensorflow / tflite-micro

Infrastructure to enable deployment of ML models to low-power resource-constrained embedded targets (including microcontrollers and digital signal processors).
Apache License 2.0
1.93k stars 827 forks source link

ci(bazel): remove redundant CI job bazel_tests_tflite_tools #2845

Closed rkuester closed 2 days ago

rkuester commented 2 days ago

Remove the redundant CI job bazel_tests_tflite_tools. It is a subset of the CI job bazel_tests.

BUG=see description

rkuester commented 2 days ago

There are a few CI cleanups coming on this branch ahead of enabling CI testing of compression. These cleanups are on a separate branch in order to make review and merging a little more parallel.

rkuester commented 2 days ago

I tried to delete a CI job in this PR, but the deleted job tried to run and failed. Does some version other than the .github/workflows directory in the commit which triggers CI govern the CI run?

rkuester commented 2 days ago

I tried to delete a CI job in this PR, but the deleted job tried to run and failed. Does some version other than the .github/workflows directory in the commit which triggers CI govern the CI run?

Yes, it appears GitHub runs the workflows in the commit into which the PR will be merged. Therefore the job and its script cannot be removed at the same time. Fixed by 55135d1.