mirror of
https://chromium.googlesource.com/crosvm/crosvm
synced 2025-02-05 18:20:34 +00:00
health-check: Add option to run only some checks
The default is still to run all of them, but this will allow us to run checks in separate luci steps to provide an easier to read result. BUG=b:233913455 TEST=./tools/health-check python fmt Change-Id: Iddc803f8f423db36ece53a13acfe564560b789a6 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/crosvm/+/3668812 Reviewed-by: Daniel Verkamp <dverkamp@chromium.org> Tested-by: kokoro <noreply+kokoro@google.com>
This commit is contained in:
parent
c2292ef52f
commit
784ab4b469
1 changed files with 18 additions and 6 deletions
|
@ -5,8 +5,9 @@
|
|||
|
||||
import doctest
|
||||
import importlib
|
||||
from pathlib import Path
|
||||
import argh # type: ignore
|
||||
import sys
|
||||
from pathlib import Path
|
||||
from impl.common import CROSVM_ROOT, run_main, cmd, chdir, parallel, find_scripts
|
||||
from impl.check_code_hygiene import has_crlf_line_endings
|
||||
|
||||
|
@ -35,13 +36,24 @@ def check_crlf_line_endings():
|
|||
sys.exit(-1)
|
||||
|
||||
|
||||
def main():
|
||||
@argh.arg("checks", choices=["python", "misc", "fmt", "clippy", "all", []])
|
||||
def main(*checks: str):
|
||||
chdir(CROSVM_ROOT)
|
||||
|
||||
check_python()
|
||||
check_crlf_line_endings()
|
||||
cmd("./tools/fmt --check").fg()
|
||||
cmd("./tools/clippy").fg()
|
||||
if not checks:
|
||||
checks = ("all",)
|
||||
|
||||
def should_check(name: str):
|
||||
return "all" in checks or name in checks
|
||||
|
||||
if should_check("python"):
|
||||
check_python()
|
||||
if should_check("misc"):
|
||||
check_crlf_line_endings()
|
||||
if should_check("fmt"):
|
||||
cmd("./tools/fmt --check").fg()
|
||||
if should_check("clippy"):
|
||||
cmd("./tools/clippy").fg()
|
||||
|
||||
|
||||
if __name__ == "__main__":
|
||||
|
|
Loading…
Reference in a new issue