ci: check for lack of byte-compilation warnings

As part of that we test with latest released Emacs version rather than
snapshot, because we don't want CI to start suddenly failing on
unrelated changes because upstream introduced a new warning/check.

Author: Hi-Angel

.github/workflows/test.yml

@@ -12,7 +12,7 @@ jobs:
           - 25.3
           - 26.3
           - 27.1
-          - snapshot
+          - 29.2
     steps:
     - uses: purcell/setup-emacs@master
       with:

Makefile

@@ -26,7 +26,9 @@ compile: $(ELCFILES)
 -include .depend
 
 $(ELCFILES): %.elc: %.el
-	$(EMACS) --batch -Q -L . --eval "(declare-function undo-redo nil)" -f batch-byte-compile $<
+	$(EMACS) --batch -Q -L . --eval "(declare-function undo-redo nil)" \
+		--eval "(setq byte-compile-error-on-warn t)" -f batch-byte-compile $<
+
 
 # Byte-compile all files in one batch. This is faster than
 # compiling each file in isolation, but also less stringent.
@@ -55,7 +57,7 @@ clean:
 # The TAG variable may specify a test tag or a test name:
 #       make test TAG=repeat
 # This will only run tests pertaining to the repeat system.
-test:
+test: compile
 	$(EMACS) -nw -Q --batch -L . -l evil-tests.el \
 		--eval "(evil-tests-initialize '(${TAG}) '(${PROFILER}))"