Skip to content

ci(api): add STREAMS_URL to turbo globalEnv#1397

Closed
Kitenite wants to merge 1 commit into
mainfrom
kitenite/fix-api-ci
Closed

ci(api): add STREAMS_URL to turbo globalEnv#1397
Kitenite wants to merge 1 commit into
mainfrom
kitenite/fix-api-ci

Conversation

@Kitenite
Copy link
Copy Markdown
Collaborator

@Kitenite Kitenite commented Feb 11, 2026

Summary

  • Add STREAMS_URL to Turbo's globalEnv so it is available during API builds in CI
  • Without this, the API build fails because the env var is defined in apps/api/src/env.ts but not passed through by Turbo

Changes

  • turbo.jsonc: Added STREAMS_URL to the globalEnv array

Test Plan

  • Verify API builds successfully in CI with STREAMS_URL available

Summary by CodeRabbit

  • Chores
    • Added STREAMS_URL as a globally available environment variable for enhanced application configuration flexibility

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Feb 11, 2026

📝 Walkthrough

Walkthrough

Adds STREAMS_URL environment variable to the global environment configuration in turbo.jsonc, making it available across the build system without requiring individual module configuration.

Changes

Cohort / File(s) Summary
Configuration
turbo.jsonc
Added STREAMS_URL to globalEnv list after KV_REST_API_TOKEN, enabling the environment variable for global access during builds.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Poem

🐰 A little variable hops into view,
STREAMS_URL joins the global crew,
Through turbo's config it finds its place,
Building with grace across the codebase! 🌊

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main change: adding STREAMS_URL to turbo's globalEnv configuration for CI builds.
Description check ✅ Passed The description is mostly complete with a clear summary, specific changes listed, and a test plan, though it doesn't follow all template sections like Related Issues or Type of Change.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch kitenite/fix-api-ci

No actionable comments were generated in the recent review. 🎉


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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 11, 2026

🚀 Preview Deployment

🔗 Preview Links

Service Status Link
Neon Database (Neon) View Branch
Fly.io Electric (Fly.io) View App
Fly.io Streams (Fly.io) View App
Vercel API (Vercel) Failed to deploy
Vercel Web (Vercel) Open Preview
Vercel Marketing (Vercel) Open Preview
Vercel Admin (Vercel) Open Preview
Vercel Docs (Vercel) Open Preview

Preview updates automatically with new commits

@Kitenite Kitenite deleted the kitenite/fix-api-ci branch February 11, 2026 19:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant