diff options
| author | Craig Jennings <c@cjennings.net> | 2026-05-05 09:50:57 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-05-05 09:50:57 -0500 |
| commit | 18bad1aa33e25873b7f30b9d42adde071a63bad9 (patch) | |
| tree | 3f4d4761680b58d7b23f26f1b9c9c8d18a0ece87 /tests/Makefile | |
| parent | 1f57189ffbe4f039ea213918c97d0a9496b4db7b (diff) | |
| download | chime-18bad1aa33e25873b7f30b9d42adde071a63bad9.tar.gz chime-18bad1aa33e25873b7f30b9d42adde071a63bad9.zip | |
refactor: clear the rest of the lint backlog and make `make lint' actually work
Took the package-lint commit's foundation and finished the job. Three
groups of changes:
1. **checkdoc** is now clean across all three package files. I worked
through every warning: 13 missing-double-space-after-period fixes, 4
message-capitalization fixes, 1 trailing-period-on-error fix, 6
Lisp-symbol-should-be-quoted fixes, and one rephrase from "starts" to
the imperative "Log the start of an async check". One `chime-debug.el'
docstring also exceeded 80 columns and got split into two lines.
2. **`make lint'** ran elisp-lint but had been silently broken. The shell
glob `test-*.el' wasn't expanding (CWD ended up at project root, not
tests/), and the noisy validators flagged everything. I rewrote the
target to lint package files explicitly via absolute paths, preload
chime.el so the byte-compiler sees cross-file symbols when checking
chime-debug.el, and disable the validators that conflict with project
style or duplicate other lint steps:
- `--no-checkdoc' (covered by `eask lint checkdoc')
- `--no-package-lint' (covered by `eask lint package'; running it on
auxiliary files re-flags them as standalone packages)
- `--no-indent-character' (project uses spaces per
`.claude/rules/elisp.md'; the validator defaults to tabs)
- `--no-fill-column' (project allows up to 80; validator defaults to 70)
- `--no-indent' (false-positives on dash threading macros `->'/`->>')
The validators-disabled list is documented in the recipe header.
3. **Generated autoload files** (`chime-autoloads.el',
`tests/tests-autoloads.el') are now gitignored. `tests/tests-autoloads.el'
was tracked from an earlier commit; I removed it from the index.
Eask regenerates these on every `eask compile' / `eask install-deps'
run, so they don't belong in the tree.
Verified: `make compile' clean (with byte-compile-error-on-warn t),
`make test-all' green at 677 tests, `eask lint package' clean,
`eask lint checkdoc' clean, `make lint' clean. Full lint backlog is now
zero across all three checks.
Diffstat (limited to 'tests/Makefile')
| -rw-r--r-- | tests/Makefile | 27 |
1 files changed, 24 insertions, 3 deletions
diff --git a/tests/Makefile b/tests/Makefile index 7cd6c0b..abf91bb 100644 --- a/tests/Makefile +++ b/tests/Makefile @@ -221,13 +221,34 @@ validate: exit 1; \ fi -# Comprehensive linting with elisp-lint (via eask-installed dev dep) +# Comprehensive linting with elisp-lint (via eask-installed dev dep). +# Scope: package files only. Tests don't pass package-lint (they aren't +# standalone packages). +# Validators disabled and why: +# - checkdoc: covered by `eask lint checkdoc' as its own MELPA-prep step. +# - package-lint: covered by `eask lint package' as its own step; running it +# here re-flags the auxiliary files because they no longer carry their own +# Package-Requires. +# - indent-character: project uses spaces (per `.claude/rules/elisp.md'); +# the validator defaults to requiring tabs. +# - fill-column: project allows up to 80 columns; the validator defaults to 70. +# - indent: produces false positives on dash threading macros (`->', `->>') +# whose indentation rule the validator doesn't know. Real indent bugs +# are caught by check-parens and review. lint: check-deps @printf "$(YELLOW)Running elisp-lint...$(NC)\n" - @$(EMACS_BATCH) --eval "(require 'elisp-lint)" \ + @$(EMACS_BATCH) \ + -l $(PROJECT_ROOT)/chime.el \ + --eval "(require 'elisp-lint)" \ -f elisp-lint-files-batch \ --no-checkdoc \ - ../chime.el test-*.el testutil-*.el 2>&1; \ + --no-package-lint \ + --no-indent-character \ + --no-fill-column \ + --no-indent \ + $(PROJECT_ROOT)/chime.el \ + $(PROJECT_ROOT)/chime-debug.el \ + $(PROJECT_ROOT)/chime-org-contacts.el 2>&1; \ if [ $$? -eq 0 ]; then \ printf "$(GREEN)✓ Linting completed successfully$(NC)\n"; \ else \ |
