branch: elpa/el-job
commit 2b52e884ccc6aeb71250741b844991471c552536
Author: Martin Edström <[email protected]>
Commit: Martin Edström <[email protected]>
Lint
---
.github/workflows/test.yml | 15 ++++++++-------
el-job.el | 2 +-
2 files changed, 9 insertions(+), 8 deletions(-)
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
index d8b17f552a..68fd626edb 100644
--- a/.github/workflows/test.yml
+++ b/.github/workflows/test.yml
@@ -61,14 +61,15 @@ jobs:
echo "SANDBOX_DIR=$SANDBOX_DIR" >> $GITHUB_ENV
./makem.sh -vv --sandbox=$SANDBOX_DIR --install-deps --install-linters
- # The "all" rule is not used, because it treats compilation warnings
- # as failures, so linting and testing are run as separate steps.
-
- - name: Lint
- # NOTE: Uncomment this line to treat lint failures as passing
- # so the job doesn't show failure.
+ # No checkdoc because I don't want to add ispell word exceptions
+ - name: Lint (sans checkdoc)
# continue-on-error: true
- run: ./makem.sh -vv --sandbox=$SANDBOX_DIR lint
+ run: ./makem.sh -vv --sandbox=$SANDBOX_DIR \
+ lint-compile \
+ lint-declare \
+ lint-indent \
+ lint-package \
+ lint-regexps
- name: Test
if: always() # Run test even if linting fails.
diff --git a/el-job.el b/el-job.el
index f2baa12dd8..b9ec7cbb8b 100644
--- a/el-job.el
+++ b/el-job.el
@@ -47,7 +47,7 @@
(require 'cl-lib)
(require 'el-job-child)
-;; Seems not to work on the GitHub Action at .github/workflows/test.yml
+;; Seems not to work on the CI at .github/workflows/test.yml
;; (declare-function native-comp-unit-file "data.c")
;; (declare-function subr-native-comp-unit "data.c")
