ci(e2e): clarify Stripe test skip message for fork PRs#1200
Conversation
The previous message ('STRIPE_TEST_SK_KEY secret not configured') was
misleading on PRs from forked repos, where the secret IS configured at
the repo level but is intentionally not exposed by GitHub Actions to
fork-PR workflow runs.
Explain that this is expected GitHub security behaviour, list when the
Stripe tests will actually run (push to main, or PRs from upstream
branches), and point to gh secret list for local verification.
|
Warning Rate limit exceeded
You’ve run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
SummaryImproves the message printed when the E2E workflow skips the Stripe tests, so WhyWhen a PR is opened from a forked repository, GitHub Actions intentionally EvidenceRun Run Both secrets are present in ChangeSingle workflow file: Verification
aidevops.sh v3.15.38 plugin for OpenCode v1.14.48 with gpt-5.5 spent 5h 32m and 15,815 tokens on this as a headless worker. Merged via PR #1200 to main. |
|
Performance Test Results Performance test results for e394589 are in 🛎️! Note: the numbers in parentheses show the difference to the previous (baseline) test run. Differences below 2% or 0.5 in absolute values are not shown. URL:
|
Summary
Improves the message printed when the E2E workflow skips the Stripe tests, so
the cause is obvious from the run log.
Why
When a PR is opened from a forked repository, GitHub Actions intentionally
does not expose repo secrets (
STRIPE_TEST_PK_KEY,STRIPE_TEST_SK_KEY) tothe workflow run. This is a security feature, not a misconfiguration.
The previous skip message ("STRIPE_TEST_SK_KEY secret not configured") implied
the secret was missing at the repo level, which led to wasted investigation
time — the secret was configured (verifiable via
gh secret list).Evidence
Run
25765984372(PR from fork
kenedytorcatt/ultimate-multisite) — Stripe step env block:Run
25762586860(push to
main) — same step:Both secrets are present in
gh secret list --repo Ultimate-Multisite/ultimate-multisite.Change
Single workflow file:
.github/workflows/e2e.yml. The skip message nowexplains the fork-PR security behaviour, lists when Stripe tests will run
(push to
main, or PRs from upstream branches), and points togh secret listfor local verification. No logic change — the same
exit 0path runs when thekey is empty.
Verification
python3 -c "import yaml; yaml.safe_load(...)".main, the Stripe step on this push event will run as before(secrets exposed), confirming the gate still passes when secrets are available.
aidevops.sh v3.15.38 plugin for OpenCode v1.14.48 with gpt-5.5 spent 5h 32m and 15,815 tokens on this as a headless worker.