Skip to content

Add a CI gate and record the deployed commit - #4

Merged
sarthakagrawal927 merged 1 commit into
mainfrom
feat/add-ci
Aug 23, 2026
Merged

Add a CI gate and record the deployed commit#4
sarthakagrawal927 merged 1 commit into
mainfrom
feat/add-ci

Conversation

@sarthakagrawal927

Copy link
Copy Markdown
Member

gitstat had no workflow files at all while serving a live Pages deployment —
nothing checked a change before it shipped. It was the only web app in the fleet
in that state.

CI

- run: npm ci --ignore-scripts
- run: npm run lint
- run: npm run build   # tsc -b && vite build — the typecheck gate too

Verified locally before adding it so main starts green, rather than landing a
gate that immediately goes red:

$ npm run lint  → exit 0 (two no-useless-escape warnings, non-fatal)
$ npm run build → ✓ built in 140ms

Concurrency group is per-ref with cancel-in-progress, matching the rest of the
fleet.

Deploy

deploy now passes --commit-hash "$(git rev-parse HEAD)" so the live
deployment's source SHA is recorded explicitly instead of inferred from whatever
the local checkout happened to be.

🤖 Generated with Claude Code

gitstat had no workflows at all despite serving a live Pages deployment, so
nothing checked a change before it shipped. Verified locally before adding the
gate: `npm run lint` exits 0 and `npm run build` succeeds, so main starts green.

`build` is `tsc -b && vite build`, which makes it the typecheck gate too.

Also passes --commit-hash on deploy so the live deployment's source SHA is
recorded explicitly rather than inferred from the local checkout.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@sarthakagrawal927
sarthakagrawal927 merged commit 95c0436 into main Aug 23, 2026
1 check passed
@sarthakagrawal927
sarthakagrawal927 deleted the feat/add-ci branch August 23, 2026 08:38
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