summaryrefslogtreecommitdiff
path: root/firmware_builder.py
diff options
context:
space:
mode:
authorJeremy Bettis <jbettis@google.com>2023-02-03 16:55:39 -0700
committerChromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com>2023-02-07 01:50:59 +0000
commita6d5bae33b7eb81d8a72f548e26c1e7678c96149 (patch)
treefd539f1ba68824bdbce31c951c12dd3b5200e540 /firmware_builder.py
parentacf34a2216b99d5ff8f11a7c106a9e929c08d847 (diff)
downloadchrome-ec-a6d5bae33b7eb81d8a72f548e26c1e7678c96149.tar.gz
cq: Print formatting commands
Print black and isort commands before running. BRANCH=None BUG=b:267757300 TEST=CQ Change-Id: I4cdd225a811effd4bf8feba20775281f9e66260a Signed-off-by: Jeremy Bettis <jbettis@google.com> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/4226458 Auto-Submit: Jeremy Bettis <jbettis@chromium.org> Reviewed-by: Eric Yilun Lin <yllin@google.com> Tested-by: Jeremy Bettis <jbettis@chromium.org> Commit-Queue: Eric Yilun Lin <yllin@google.com>
Diffstat (limited to 'firmware_builder.py')
-rwxr-xr-xfirmware_builder.py24
1 files changed, 13 insertions, 11 deletions
diff --git a/firmware_builder.py b/firmware_builder.py
index 2fcccd6476..d633f56388 100755
--- a/firmware_builder.py
+++ b/firmware_builder.py
@@ -65,19 +65,21 @@ def build(opts):
ec_dir = pathlib.Path(__file__).parent
# Run formatting checks on all python files.
- subprocess.run(
- ["black", "--check", "."], cwd=os.path.dirname(__file__), check=True
- )
+ cmd = ["black", "--check", "."]
+ print(f"# Running {' '.join(cmd)}.")
+ subprocess.run(cmd, cwd=os.path.dirname(__file__), check=True)
chromite_dir = ec_dir.resolve().parent.parent.parent / "chromite"
+ cmd = [
+ "isort",
+ f"--settings-file={chromite_dir / '.isort.cfg'}",
+ "--check",
+ "--gitignore",
+ "--dont-follow-links",
+ ".",
+ ]
+ print(f"# Running {' '.join(cmd)}.")
subprocess.run(
- [
- "isort",
- f"--settings-file={chromite_dir / '.isort.cfg'}",
- "--check",
- "--gitignore",
- "--dont-follow-links",
- ".",
- ],
+ cmd,
cwd=os.path.dirname(__file__),
check=True,
)