branch: elpa/loopy-dash
commit d98d1a0a7b2d97a5a3517a7e3f5cc941d2331413
Author: okamsn <[email protected]>
Commit: okamsn <[email protected]>
Update Forgejo Actions workflow. (#10)
- Don't trigger the workflow on pushes to the main branch, as
(1) we don't allow pushing to the main branch and (2) this
triggers after merging a PR for which we've already run the checks.
This was done for Loopy in https://codeberg.org/okamsn/loopy/pulls/294
- Comment out the `permissions` key, which is not currently used by Forgejo
Actions. The warning displayed in the job runner UI recommends using
"Authorized Integrations":
https://forgejo.org/docs/latest/user/authorized-integrations/
Reviewed-on: https://codeberg.org/okamsn/loopy-dash/pulls/10
---
.forgejo/workflows/emacs-matrix-tests.yml | 17 ++++++++++++-----
1 file changed, 12 insertions(+), 5 deletions(-)
diff --git a/.forgejo/workflows/emacs-matrix-tests.yml
b/.forgejo/workflows/emacs-matrix-tests.yml
index d39f8cedf4..8b9bbb33bd 100644
--- a/.forgejo/workflows/emacs-matrix-tests.yml
+++ b/.forgejo/workflows/emacs-matrix-tests.yml
@@ -1,14 +1,21 @@
name: Test Emacs Matrix
on:
- push:
- branches:
- - main
- - master
+ # The `push` event also triggers after merging pull requests that have
+ # already been checked, which is redundant.
+ #
+ # push:
+ # branches:
+ # - main
+ # - master
pull_request:
branches:
- main
- master
-permissions: {}
+# `permissions` is not used by Forgejo Actions.
+# Forgejo's warning recommends using "Authorized Integrations":
+# https://forgejo.org/docs/latest/user/authorized-integrations/
+#
+# permissions: {}
jobs:
test:
name: 'Install and Test'
