History

ci: skip E2E (stdio MCP) on pull_request; scope marketing lint override to .ts (#1220)

The local E2E scenario boots a real executor web plus a browser and is currently flaky on PRs, so gate it to push events (still runs on main). The marketing app is plain Astro/Cloudflare SSR, not Effect domain code, but its lib/*.ts files were subject to the Effect-only lint rules (no-try-catch-or-throw, no-promise-catch), breaking lint on main after the GitHub stars pill landed. Extend the existing marketing override from *.astro to *.{astro,ts,tsx} and disable no-promise-catch there too.

Rhys Sullivan committed 4 months ago

Commit 7c47b031 parent bb0a02b2 files · +5 −1Browse files

Showing the changes to .github (1 of 2 files). Show all

Unified · Split

.github/workflows/ci.yml+3−0
@@ -76,6 +76,9 @@ jobs: · 75 unmodified lines
7676
7777 e2e-local:
7878 name: E2E (stdio MCP)
79+ # Skipped on pull_request: the local scenario boots a real `executor web`
80+ # plus a browser and is currently flaky on PRs. Still runs on push to main.
81+ if: github.event_name != 'pull_request'
7982 runs-on: ubuntu-latest
8083 steps:
8184 - uses: actions/checkout@v4