Skip to content

Commit 1f1ccc4

Browse files
committed
docs: clarify why InfraLens e2e stays single-worker in CI
Serialized because this script also runs locally where a developer may have real Upstash credentials, not because CI itself is rate-limited — CI never has credentials, so the limiter runs allow-all there.
1 parent 691daa4 commit 1f1ccc4

1 file changed

Lines changed: 7 additions & 2 deletions

File tree

.github/workflows/ci.yml

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -115,8 +115,13 @@ jobs:
115115
- run: pnpm install --frozen-lockfile
116116
- run: pnpm exec playwright install --with-deps chromium webkit
117117
- run: pnpm build
118-
# Rate-limited (1 request/IP/30s) — must stay on a single worker,
119-
# see playwright.config.ts and migration plan §17/§28.
118+
# Serialized (--workers=1, see playwright.config.ts) — not because CI
119+
# itself is rate-limited (it never has Upstash credentials, so
120+
# src/lib/rate-limit runs allow-all here), but because this same
121+
# script also runs locally, where a developer may have added real
122+
# credentials and would hit the genuine InfraLens policy. Staying
123+
# serialized keeps that case safe without needing to know which one
124+
# is active.
120125
- run: pnpm e2e:infralens
121126
env:
122127
CI: true

0 commit comments

Comments
 (0)