fix(workflow): read GRAPHRAG_UI_URL from secrets, not vars#480
fix(workflow): read GRAPHRAG_UI_URL from secrets, not vars#480galshubeli wants to merge 1 commit into
Conversation
The repo has ``GRAPHRAG_UI_URL`` set as a GitHub Actions secret, so
``${{ vars.GRAPHRAG_UI_URL }}`` resolved to an empty string and curl
failed with ``URL rejected: No host part in the URL`` on the first
real PR merge to main.
A public URL doesn't strictly need to be in the secret namespace
(variables are fine and visible in the UI for debugging), but
matching the workflow to where the value already lives is the
least-disruptive fix.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
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. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
✨ 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 |
|
Closing — is now set as a repo variable (the cleaner approach), so the workflow on main reads it correctly as-is. Variable: |
1 participant
Summary
First real merge to main hit
curl: (3) URL rejected: No host part in the URLbecause the workflow reads${{ vars.GRAPHRAG_UI_URL }}but the repo has the URL configured as a secret, not a variable, so the env var resolved to an empty string.A public URL doesn't strictly need to live in the secret namespace (variables are fine and visible in the UI for debugging), but matching the workflow to where the value already exists is the least-disruptive fix.
Change
One line in
.github/workflows/update-graph.yml:After merge
The next
.mdpush to main should run the workflow successfully end-to-end. Result will mirror the wet-run done earlier today (docs_v6_*pending → smoke test → atomic alias flip on:Graphnode).🤖 Generated with Claude Code