Skip to content

Conversation

@brojd
Copy link
Contributor

@brojd brojd commented Feb 10, 2026

No description provided.

Copilot AI review requested due to automatic review settings February 10, 2026 05:40
@vercel
Copy link

vercel bot commented Feb 10, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
archestra Ignored Ignored Feb 10, 2026 6:06am

Request Review

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the QuickStart UI snippet related to ngrok environment configuration in QuickStartBlock, intended to reflect a renamed env var.

Changes:

  • Introduces ARCHESTRA_NGROK_AUTH_TOKEN into the quickstart flow output.
  • Updates the rendered docker run snippet (but leaves existing NGROK_API_KEY usage in place).
Comments suppressed due to low confidence (1)

app/components/QuickStartBlock.tsx:192

  • The rendered docker command now includes the literal text ARCHESTRA_NGROK_AUTH_TOKEN on the line that should start with -e ..., which makes the displayed command confusing/invalid. If the intent is to rename the ngrok env var, update the command generation/rendering consistently (the component still uses NGROK_API_KEY elsewhere); otherwise remove this extra token.
          <span className="text-gray-300">ARCHESTRA_NGROK_AUTH_TOKEN
            {'  '}-e <span className="text-purple-400">ARCHESTRA_QUICKSTART</span>=true
          </span>{' '}
          {renderLineContinuation()}
          {'\n'}

@brojd brojd merged commit c4c21fb into main Feb 10, 2026
11 checks passed
@brojd brojd deleted the brojd/update-ngrok-env-var-name branch February 10, 2026 06:07
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