pw_presubmit: Add static_analysis to combined step
Add the static_analysis ninja target to the gn_combined_build_check
presubmit step.
Bug: b/243380637
Change-Id: I2af79d1205ca8e457aee19df1d71b448af859c62
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/107930
Reviewed-by: Wyatt Hepler <hepler@google.com>
Commit-Queue: Rob Mohr <mohrr@google.com>
diff --git a/pw_presubmit/py/pw_presubmit/pigweed_presubmit.py b/pw_presubmit/py/pw_presubmit/pigweed_presubmit.py
index 2e1e7c2..7105647 100755
--- a/pw_presubmit/py/pw_presubmit/pigweed_presubmit.py
+++ b/pw_presubmit/py/pw_presubmit/pigweed_presubmit.py
@@ -143,6 +143,7 @@
@_BUILD_FILE_FILTER.apply_to_check()
def gn_combined_build_check(ctx: PresubmitContext) -> None:
+ """Run most host and device (QEMU) tests."""
build_targets = [
*_at_all_optimization_levels('stm32f429i'),
*_at_all_optimization_levels(f'host_{_HOST_COMPILER}'),
@@ -158,6 +159,11 @@
build_targets.append('cpp14_compatibility')
build_targets.append('cpp20_compatibility')
+ # clang-tidy doesn't run on Windows.
+ if sys.platform != 'win32':
+ build_targets.append('static_analysis')
+
+ # QEMU doesn't run on Windows.
if sys.platform != 'win32':
build_targets.extend(_at_all_optimization_levels('qemu_gcc'))
build_targets.extend(_at_all_optimization_levels('qemu_clang'))