Closed chiphogg closed 3 weeks ago
@timhirsh, see this failed build. Note that evidently, when one matrix build fails, it cancels the others. This seems OK, since they all need to be passing all the time.
Now instead of getting 11 notifications from the GitHub app, every time I make, update, or land a PR... I can get 13! :laughing:
Currently, we only run one CMake configuration in CI. It's the default one, where we permit the GTest dependency, and we build and run tests. Now we are adding two more configurations, one for each of the new options:
AU_ENABLE_TESTING
toFALSE
AU_EXCLUDE_GTEST_DEPENDENCY
toTRUE
(implies the former)In both these instances, the
test
target won't exist, so we exclude it from running.We do however retain the
CMAKE_VERIFY_INTERFACE_HEADER_SETS
option, because that gives us useful testing for our interface targets.Fixes #304.