Skip to content

ci: expand workflow triggers and test Windows desktop dependencies - #10

Merged
llm-learner merged 1 commit into
llm-learner:mainfrom
lska367:codex/github-actions-upstream
Sep 26, 2026
Merged

llm-learner merged 1 commit into
llm-learner:mainfrom
lska367:codex/github-actions-upstream

Conversation

@lska367

@lska367 lska367 commented Sep 25, 2026

Copy link
Copy Markdown
Contributor

What / 改了什么

Extend the existing test workflow to run on branch pushes as well as PRs and manual dispatch. Keep the 12 Linux/Windows/macOS × Python 3.10–3.13 combinations, and rerun the Windows Python 3.13 suite with desktop dependencies installed.

Add a 15-minute job timeout, cancel superseded runs, use the selected Python for dashboard tests, enable UTF-8, update the setup actions to v6, and pin the tested Node.js major to 22. Keep read-only repository permissions and disable checkout credential persistence. Update both READMEs.

Why / 为什么

Branch changes currently receive no push checks, and the optional Windows tray tests are skipped without their dependencies. This adds that coverage while keeping the standard-library-only run. Native tray and WebView2 smoke tests remain opt-in locally.

How was it tested? / 如何测试

Local macOS, Python 3.11.9, Node.js 22.23.2:

  • python -m unittest discover -s skills/headroom/tests -p "test_*.py" -v with HEADROOM_DISABLE_DASHBOARD=1, HEADROOM_TEST_PYTHON=python3, and PYTHONUTF8=1: 70 passed, 10 skipped.
  • node skills/headroom/tests/test_dashboard_mood.js: passed.
  • actionlint 1.7.12 and git diff --check: passed.

The hosted matrix and Windows desktop dependency run have not yet been verified.

Checklist / 检查项

  • Core code stays standard-library only; no product code or PowerShell changes.
  • No prompt text, history, ledgers, or credentials committed.
  • README.md and README.zh-CN.md updated together.

@llm-learner
llm-learner merged commit dd9afea into llm-learner:main Sep 26, 2026
12 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.

2 participants