Should driver rebuild non-test targets? about target-determinator HOT 1 OPEN

Non-test targets are correctly identified by target_determinator but appear skipped by the driver.

They should end up being built; if you bazel test //some:binary //some:test, bazel should build both targets, but only run the second test - your build should fail if //some:binary fails to build. Do you have an example you can share where this isn't the case?

I wonder if it would be better to split the targets by build / test targets.

In general I wouldn't recommend that, as you'll lose parallelism - some of the build and test could otherwise be done in parallel, and now will be forced to be serialised.

For our CI, I'm thinking of using target_determinator to get all the targets, then writing our own driver that does both build / test (not just test).

target-determinator was very intentionally written as a building block to make it easy to integrate into your own custom driver, because in my experience most organisations have subtly different desires from their workflows - honestly the bundled driver was meant more as an example of how to do this rather than something most people would pick up off the shelf, but in the hopes that it's sufficient for many smaller organisations' use-cases without needing further customisation. If it's broken in some way (and I would count "doesn't fail in the case of build failures" as broken), I'd be very happy to write or review a fix, but it's also intended that folks can build their own more complex logic around it if they prefer :)

from target-determinator.