Skip to content

Fixes vite launch config env variables#278753

Merged
hediet merged 1 commit intomainfrom
hediet/b/fiscal-unicorn
Nov 21, 2025
Merged

Fixes vite launch config env variables#278753
hediet merged 1 commit intomainfrom
hediet/b/fiscal-unicorn

Conversation

@hediet
Copy link
Member

@hediet hediet commented Nov 21, 2025

No description provided.

Copilot AI review requested due to automatic review settings November 21, 2025 11:38
@hediet hediet enabled auto-merge (squash) November 21, 2025 11:38
@hediet hediet self-assigned this Nov 21, 2025
@vs-code-engineering vs-code-engineering bot added this to the November 2025 milestone Nov 21, 2025
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

This PR fixes environment variable configuration for the Vite-based launch configuration used for hot reload development. It ensures that the necessary environment variables are set for the development server to function correctly and suppresses an irrelevant Vite warning.

  • Adds VSCODE_DEV_SERVER_URL and VSCODE_DEV environment variables to the "Launch VS Code Internal (Hot Reload)" configuration
  • Suppresses a Vite warning for dynamic imports that cannot be analyzed but are not problematic

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
.vscode/launch.json Adds VSCODE_DEV_SERVER_URL and VSCODE_DEV environment variables to the hot reload launch configuration
build/vite/vite.config.ts Suppresses Vite warning for dynamic import statement that cannot be statically analyzed

@hediet hediet merged commit 2b1a433 into main Nov 21, 2025
33 of 34 checks passed
@hediet hediet deleted the hediet/b/fiscal-unicorn branch November 21, 2025 13:12
@vs-code-engineering vs-code-engineering bot locked and limited conversation to collaborators Jan 5, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants