Skip to content

Commit 8f04d8a

Browse files
committed
ci: drop e2e_scan from PR-blocking matrix (live public-API dependency)
`e2e_scan` exercises `socket-patch scan --apply --yes` end-to-end against the live public proxy at patches-api.socket.dev. The proxy returns "Service temporarily over capacity" intermittently for `/patch/by-package/*` lookups, and the test panics when scan reports zero patches. Move it out of the PR matrix; runnable on demand via `cargo test -p socket-patch-cli --test e2e_scan -- --ignored`. The other matrix entries (e2e_npm, e2e_pypi, e2e_cargo, e2e_golang, e2e_maven, e2e_gem, e2e_composer, e2e_nuget) stay — they exercise local crawler / apply paths, not the live API. Assisted-by: Claude Code:claude-opus-4-7
1 parent fbd5f23 commit 8f04d8a

1 file changed

Lines changed: 8 additions & 4 deletions

File tree

.github/workflows/ci.yml

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -407,10 +407,14 @@ jobs:
407407
suite: e2e_npm
408408
- os: macos-latest
409409
suite: e2e_pypi
410-
- os: ubuntu-latest
411-
suite: e2e_scan
412-
- os: macos-latest
413-
suite: e2e_scan
410+
# `e2e_scan` is intentionally NOT in the PR matrix — it
411+
# depends on the live public proxy at
412+
# patches-api.socket.dev serving real patch data for
413+
# `minimist@1.2.2`, which is subject to rate-limiting
414+
# ("Service temporarily over capacity") and other
415+
# availability quirks outside this repo's control. Run on
416+
# demand with
417+
# `cargo test -p socket-patch-cli --test e2e_scan -- --ignored`.
414418
# Safety-hardening e2e suites. The fast non-ignored ones
415419
# (e2e_safety_lock, e2e_safety_yarn_pnp) run via the
416420
# standard `test` job above on all three platforms, so no

0 commit comments

Comments
 (0)