Skip to content

Fix CI test failures (Emacs 28 compat, state pollution, log visibility)#22

Merged
ywatanabe1989 merged 1 commit into
mainfrom
develop
Mar 21, 2026
Merged

Fix CI test failures (Emacs 28 compat, state pollution, log visibility)#22
ywatanabe1989 merged 1 commit into
mainfrom
develop

Conversation

@ywatanabe1989

Copy link
Copy Markdown
Owner

Summary

  • Fix string-empty-p availability on Emacs 28 (needs subr-x)
  • Fix global state pollution in tab-highlight test
  • Make ERT output visible in GitHub Actions logs

Test plan

  • 258/258 tests pass locally
  • ERT output now visible in CI logs via | tee pipe fix

🤖 Generated with Claude Code

- Add (require 'subr-x) for Emacs 28 compat (string-empty-p)
- Fix tab-highlight test global state pollution from pulse timer
- Fix run_tests.sh pipe to show ERT output in CI logs

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@ywatanabe1989 ywatanabe1989 merged commit 744fb5f into main Mar 21, 2026
2 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant