Skip to content

Move the solution file to root to fix issues with C# devkit#83

Merged
mattwar merged 2 commits into
mainfrom
src/mattwar/MoveSolutionFileToRoot
Apr 16, 2026
Merged

Move the solution file to root to fix issues with C# devkit#83
mattwar merged 2 commits into
mainfrom
src/mattwar/MoveSolutionFileToRoot

Conversation

@mattwar

@mattwar mattwar commented Apr 16, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings April 16, 2026 01:35

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Moves the Visual Studio solution (.slnx) to the repository root to improve compatibility with the C# Dev Kit / Visual Studio tooling, and adjusts related project paths accordingly.

Changes:

  • Relocated KustoExplorerVscode.slnx from src/ to the repo root and updated all referenced paths.
  • Removed the old src/KustoExplorerVscode.slnx.
  • Added a Client-scoped .npmrc enabling legacy-peer-deps during npm installs.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
src/KustoExplorerVscode.slnx Removed the old solution file from src/ after moving it to the repo root.
KustoExplorerVscode.slnx New root-level solution file with updated relative paths to projects and repo files.
src/Client/.npmrc Configures npm installs for the client to use legacy-peer-deps=true.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread KustoExplorerVscode.slnx
@mattwar mattwar merged commit 506e3e1 into main Apr 16, 2026
5 checks passed
@mattwar mattwar deleted the src/mattwar/MoveSolutionFileToRoot branch April 16, 2026 01:48
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.

2 participants