Skip to content

Commit f2cad3e

Browse files
committedJun 27, 2023
Install buttercup through make task, not CI
This way running `make` locally will ensure that buttercup's installed, rather than only installing it as part of the CI process.
1 parent 2a3ff70 commit f2cad3e

File tree

2 files changed

+4
-5
lines changed

2 files changed

+4
-5
lines changed
 

‎.github/workflows/test.yml

+2-5
Original file line numberDiff line numberDiff line change
@@ -38,13 +38,10 @@ jobs:
3838
with:
3939
version: ${{ matrix.emacs_version }}
4040

41-
- name: Install buttercup package
42-
run: emacs --quick --batch -f package-initialize --eval "(add-to-list 'package-archives '(\"nongnu\" . \"https://elpa.nongnu.org/nongnu/\") t)" --eval "(package-refresh-contents)" --eval "(package-install 'buttercup)"
43-
4441
- name: Run tests
4542
if: matrix.allow_failure != true
46-
run: 'make && make test'
43+
run: 'make test'
4744

4845
- name: Run tests (allow failure)
4946
if: matrix.allow_failure == true
50-
run: 'make && make test || true'
47+
run: 'make test || true'

‎Makefile

+2
Original file line numberDiff line numberDiff line change
@@ -2,5 +2,7 @@
22
test: engine-mode.el engine-mode-test.el
33
emacs --quick --batch \
44
-f package-initialize \
5+
--eval "(add-to-list 'package-archives '(\"nongnu\" . \"https://elpa.nongnu.org/nongnu/\") t)" \
6+
--eval "(when (not (package-installed-p 'buttercup)) (package-refresh-contents) (package-install 'buttercup))" \
57
-L . \
68
-f buttercup-run-discover

0 commit comments

Comments
 (0)
Please sign in to comment.