build: add golang checkstyle for hs-test to ci checkstyle-test target 37/42237/7
authorDave Wallace <[email protected]>
Thu, 23 Jan 2025 03:43:33 +0000 (22:43 -0500)
committerFlorin Coras <[email protected]>
Thu, 23 Jan 2025 17:30:24 +0000 (17:30 +0000)
- Ensure extras/hs-test code is verified for style compliance in
  the vpp-checkstyle-verify-*-*-* ci jobs

Type: make

Change-Id: If3900cbc028f0bd38b14fb9dd08841e21fec15da
Signed-off-by: Dave Wallace <[email protected]>
Makefile
extras/hs-test/Makefile

index 8ee12bc..55731dc 100644 (file)
--- a/Makefile
+++ b/Makefile
@@ -666,10 +666,13 @@ test-wipe-all:
        @$(MAKE) -C test wipe-all
 
 # Note: All python venv consolidated in test/Makefile, test/requirements*.txt
+#       Also, this target is used by ci-management/jjb/scripts/vpp/checkstyle-test.sh,
+#       thus inclusion of checkstyle-go here to include checkstyle for hs-test
+#       in the vpp-checkstyle-verify-*-*-* jobs
 .PHONY: test-checkstyle
 test-checkstyle:
-       $(warning test-checkstyle is deprecated. Running checkstyle-python.")
        @$(MAKE) -C test checkstyle-python-all
+       @$(MAKE) -C extras/hs-test checkstyle-go
 
 # Note: All python venv consolidated in test/Makefile, test/requirements*.txt
 .PHONY: test-checkstyle-diff
index ccccf4f..9fbb21c 100644 (file)
@@ -200,14 +200,14 @@ checkstyle-go: .goimports.ok
        if [ $$status -ne 0 ]; then \
                exit $$status; \
     elif [ -z "$$output" ]; then \
-        echo "*******************************************************************"; \
-        echo "Checkstyle OK."; \
-        echo "*******************************************************************"; \
+        echo "******************************************************************************"; \
+        echo "* HST Golang Checkstyle OK."; \
+        echo "******************************************************************************"; \
     else \
         echo "$$output"; \
-        echo "*******************************************************************"; \
-        echo "Checkstyle failed. Use 'make fixstyle-go' or fix errors manually."; \
-        echo "*******************************************************************"; \
+        echo "******************************************************************************"; \
+        echo "* HST Golang Checkstyle FAILED. Use 'make fixstyle-go' or fix errors manually."; \
+        echo "******************************************************************************"; \
         exit 1; \
     fi