From 045f3d7bc04c6ae8d112b87ad2ec6c63f7a26c39 Mon Sep 17 00:00:00 2001 From: joey0320 Date: Sat, 14 Jan 2023 15:38:43 -0800 Subject: [PATCH] Fix common.mk && add makefile check as pre-commit hook --- .pre-commit-config.yaml | 4 ++++ common.mk | 2 +- 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml index d498045e..5cd4ef45 100644 --- a/.pre-commit-config.yaml +++ b/.pre-commit-config.yaml @@ -9,3 +9,7 @@ repos: - id: check-yaml - id: check-json - id: check-added-large-files + - repo: https://github.com/mrtazz/checkmake.git + rev: main + hooks: + - id: checkmake diff --git a/common.mk b/common.mk index 701fdd38..17f7a8e0 100644 --- a/common.mk +++ b/common.mk @@ -158,7 +158,7 @@ SFC_REPL_SEQ_MEM = --infer-rw --repl-seq-mem -c:$(MODEL):-o:$(SFC_SMEMS_CONF) # "ENABLE_CUSTOM_FIRRTL_PASS" variable. # # hack: lower to low firrtl if Fixed types are found -# hack: when using dontTouch, io.cpu annotations are not removed by SFC, +# hack: when using dontTouch, io.cpu annotations are not removed by SFC, # hence we remove them manually by using jq before passing them to firtool $(SFC_MFC_TARGETS) &: $(FIRRTL_FILE) $(FINAL_ANNO_FILE) $(VLOG_SOURCES) ifeq (,$(ENABLE_CUSTOM_FIRRTL_PASS))