CI: warnings as errors for safety

dzid26 · Apr 9, 2024 · 39ad0b5 · 39ad0b5
1 parent 2e596ff
commit 39ad0b5
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/.github/workflows/build.yaml b/.github/workflows/build.yaml
@@ -53,7 +53,7 @@ jobs:
         run: |
           cd src/controller
           make clean
-          make
+          make CFLAGS=--Werror
       - uses: actions/upload-artifact@v3
         with:
           name: firmware

diff --git a/src/controller/Makefile b/src/controller/Makefile
@@ -65,9 +65,9 @@ ebike_app.h pins.h eeprom.h lights.h config.h
 # The list of .rel files can be derived from the list of their source files
 RELS = $(addprefix $(ODIR)/,$(notdir $(EXTRASRCS:.c=.rel)))
 
-INCLUDES = -I$(IDIR) -I$(IDIR1)
-CFLAGS = -m$(PLATFORM) -D$(DEVICE) -Ddouble=float --std-c99 --nolospre --opt-code-speed --peep-asm --peep-file peep.txt
-DEBUG_FLAGS = --out-fmt-ihx --debug
+override INCLUDES += -I$(IDIR) -I$(IDIR1)
+override CFLAGS += -m$(PLATFORM) -D$(DEVICE) -Ddouble=float --std-c99 --nolospre --opt-code-speed --peep-asm --peep-file peep.txt
+override DEBUG_FLAGS += --out-fmt-ihx --debug
 LIBS = 
 
 # This just provides the conventional target name "all"; it is optional