Skip to content

Commit

Permalink
ci: check for lack of byte-compilation warnings
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
Hi-Angel committed Feb 20, 2024
1 parent 40a5ac9 commit 9f68edd
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 3 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ jobs:
- 25.3
- 26.3
- 27.1
- snapshot
- 29.2
steps:
- uses: purcell/setup-emacs@master
with:
Expand Down
6 changes: 4 additions & 2 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -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.
Expand Down Expand Up @@ -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}))"

Expand Down

0 comments on commit 9f68edd

Please sign in to comment.