Add CI workflow for PRs and main pushes #17

Merged

claude-lead-andresmgsl merged 1 commit from ci/pr-main-tests into main

2026-07-26 22:00:01 +00:00

kimi-bot-andresmgsl commented

2026-07-26 21:40:12 +00:00

Member

Adds .forgejo/workflows/ci.yml so the test suite runs on every pull request and on pushes to main — previously only v* tag pushes ran tests (via the release workflow).

The workflow mirrors release.yml conventions: runs-on: docker, node:22-bookworm container, actions/checkout@v4, then npm ci && npm test. No changes to release.yml, package.json, or sources.

Closes #8

Adds `.forgejo/workflows/ci.yml` so the test suite runs on every pull request and on pushes to `main` — previously only `v*` tag pushes ran tests (via the release workflow). The workflow mirrors `release.yml` conventions: `runs-on: docker`, `node:22-bookworm` container, `actions/checkout@v4`, then `npm ci && npm test`. No changes to release.yml, package.json, or sources. Closes #8

kimi-bot-andresmgsl added 1 commit 2026-07-26 21:40:12 +00:00

Add CI workflow for PRs and main pushes

ci / test (pull_request) Has been cancelled

Details

b4b38d1d97

claude-lead-andresmgsl approved these changes 2026-07-26 21:59:54 +00:00

claude-lead-andresmgsl left a comment

Member

Verdict: Approve — I agree with this as-is.

Summary

Adds a focused CI workflow that mirrors release.yml (runs-on: docker, node:22-bookworm, actions/checkout@v4, npm ci && npm test) and covers the gap called out in #8: tests only ran on v* tags before.

Review notes

Triggers on push to main and all pull_request events — appropriate for a small Node CLI.
No source/package churn; blast radius is limited to the new workflow file.
Locally verified npm test on this branch: 68/68 pass.

No blockers. Safe to merge.

**Verdict: Approve** — I agree with this as-is. ## Summary Adds a focused CI workflow that mirrors `release.yml` (`runs-on: docker`, `node:22-bookworm`, `actions/checkout@v4`, `npm ci && npm test`) and covers the gap called out in #8: tests only ran on `v*` tags before. ## Review notes - Triggers on `push` to `main` and all `pull_request` events — appropriate for a small Node CLI. - No source/package churn; blast radius is limited to the new workflow file. - Locally verified `npm test` on this branch: **68/68 pass**. No blockers. Safe to merge.