Skip to content

ci: add GitHub Actions workflow (npm ci + node test/run.mjs)#4

Merged
bdelanghe merged 1 commit into
mainfrom
ci/add-workflow
Jun 28, 2026
Merged

ci: add GitHub Actions workflow (npm ci + node test/run.mjs)#4
bdelanghe merged 1 commit into
mainfrom
ci/add-workflow

Conversation

@bdelanghe

Copy link
Copy Markdown
Contributor

What & why

The kit had no CI, so every PR showed "no checks reported" and test/run.mjs was only ever run locally. This adds a real check to every kit PR (including the open conformance follow-ups).

Change

Minimal workflow, matching lone's house style (pinned action SHAs + version comments):

  • Triggers: push to main, pull_request, workflow_dispatch.
  • Node 22.x with npm cache → npm cinpm test (node test/run.mjs).

Verification

Ran exactly as CI will, locally: npm ci && npm test12 passed, 0 failed.

This is the workflow's own first run — it validates itself on this PR.

PR checklist

  1. Independent PR — only the CI workflow file
    • Verified: 1 file (.github/workflows/ci.yml)
  2. Changed codepaths verifiednpm ci && npm test green locally; self-validates on this PR
  3. Root cause identified — no workflow existed → no checks
    • Added one
  4. No duplication — N/A
  5. No unrelated changes
    • N/A

🤖 Generated with Claude Code

The kit had no CI, so every PR showed "no checks reported" and the test suite was
only ever run locally. Add a minimal workflow — pinned actions (SHA + version
comment, matching lone's house style), Node 22.x with npm cache, `npm ci` then
`npm test` — on push to main, pull_request, and manual dispatch.

Verified locally: `npm ci && npm test` → 12 passed, 0 failed.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@bdelanghe bdelanghe marked this pull request as draft June 28, 2026 21:53
@bdelanghe bdelanghe marked this pull request as ready for review June 28, 2026 21:53
@bdelanghe bdelanghe merged commit 64425d8 into main Jun 28, 2026
1 check passed
@bdelanghe bdelanghe deleted the ci/add-workflow branch June 28, 2026 23:09
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