feat(makefile): adds a make formatter as task runner

fix(ci): cpp clang-tidy now properly reads non-build files only
2026-10-30 02:22:27 +01:00 · 2026-10-30 02:19:30 +01:00
2 changed files with 16 additions and 10 deletions
@@ -40,5 +40,7 @@ jobs:
              run: |
                  sed -i 's/-mno-direct-extern-access //g' build/compile_commands.json
                  set -o pipefail
-                  run-clang-tidy -p build -j $(nproc) -header-filter="Plugins/.*" 2>&1 | \
-                    grep -v 'Suppressed\|non-user\|non-system\|unknown argument.*mno-direct\|Found compiler error\|warnings generated\.\|^Use -header-filter'
+                  run-clang-tidy -p build -j "$(nproc)" \
+                      -source-filter="^$(pwd)/Plugins/" \
+                      -header-filter=".*Plugins/.*" \
+                      -removed-arg=-mno-direct-extern-access
@@ -1,13 +1,13 @@
-BUILD		?=		cmake -B build -G Ninja
-NONUITKA ?=	cmake -S . -B build -G Ninja -DCMAKE_BUILD_TYPE=Release -DENABLE_MODULES="plugin;shell;m3shapes"
-NINJA		?=		ninja -C build
-COMPILE		?=		$(BUILD) && $(NINJA)
-INSTALL		?=		$(BUILD) && $(NINJA) && sudo ninja -C build install
+BUILD ?= cmake -B build -G Ninja
+NONUITKA ?= cmake -S . -B build -G Ninja -DCMAKE_BUILD_TYPE=Release -DENABLE_MODULES="plugin;shell;m3shapes"
+NINJA ?= ninja -C build
+COMPILE ?= $(BUILD) && $(NINJA)
+INSTALL ?= $(BUILD) && $(NINJA) && sudo ninja -C build install

-CPPFMT ?=		find . \( -name '*.cpp' -o -name '*.h' -o -name '*.hpp' \) -not -path './build/*' -exec clang-format -i --style=file {} +
-CPPLINT ?=		run-clang-tidy -p build -j "$$(nproc)" -source-filter="^$$(pwd)/Plugins/" -header-filter=".*Plugins/.*" -removed-arg=-mno-direct-extern-access
+CPPFMT ?= find . \( -name '*.cpp' -o -name '*.h' -o -name '*.hpp' \) -not -path './build/*' -exec clang-format -i --style=file {} +
+CPPLINT ?= run-clang-tidy -p build -j "$$(nproc)" -source-filter="^$$(pwd)/Plugins/" -header-filter=".*Plugins/.*" -removed-arg=-mno-direct-extern-access

-NUITKA ?=		nuitka --module --include-package=zshell cli/src/zshell/ --output-dir=cli/
+NUITKA ?= nuitka --module --include-package=zshell cli/src/zshell/ --output-dir=cli/

 .DEFAULT_GOAL := help

@@ -32,6 +32,7 @@ help:
 	@echo "check-cpp		runs all C++ checks"
 	@echo "check-py		runs all Python checks"
 	@echo "check-rs		runs all Rust checks"
+	@echo "fmt-make		formats the makefile using mbake (in case you like spaces instead of tabs)"

 build:
 	$(BUILD)
@@ -104,3 +105,6 @@ check-py:
 	$(MAKE) py-test
 	$(MAKE) py-check
 	$(MAKE) py-compile
+
+fmt-make:
+	mbake format makefile
Author	SHA1	Message	Date
AramJonghu	355117fd92	feat(makefile): adds a make formatter as task runner C++ / fmt (pull_request) Successful in 5s Details C++ / build & lint (pull_request) Failing after 10s Details JS/TS / fmt & lint (pull_request) Successful in 11s Details Python / fmt & lint (pull_request) Successful in 28s Details Python / test & nuitka build (pull_request) Failing after 1m23s Details Rust / build, clippy & fmt (pull_request) Failing after 5m10s Details	2026-10-30 02:22:27 +01:00
AramJonghu	a9b7e10b6c	fix(ci): cpp clang-tidy now properly reads non-build files only	2026-10-30 02:19:30 +01:00