From 9f68edd214bd9fe969541a543fe71a7949e7bc70 Mon Sep 17 00:00:00 2001
From: Konstantin Kharlamov <Hi-Angel@yandex.ru>
Date: Sun, 28 Jan 2024 03:25:15 +0300
Subject: [PATCH] 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.
---
 .github/workflows/test.yml | 2 +-
 Makefile                   | 6 ++++--
 2 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
index e911e9a8..c22b312d 100644
--- a/.github/workflows/test.yml
+++ b/.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:
diff --git a/Makefile b/Makefile
index 755845a0..e921c5e7 100644
--- a/Makefile
+++ b/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}))"