Skip to content

Conversation

@rajkumar-rangaraj
Copy link
Member

  • Deleted Everything.sln
  • Moved SHIM\Microsoft.ApplicationInsights.Shim.sln to Everything.sln
  • This is a 3.x and no need to call it as shim anymore

Copilot AI review requested due to automatic review settings December 18, 2025 03:45
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 consolidates solution files by replacing the root Everything.sln with the content from SHIM/Microsoft.ApplicationInsights.Shim.sln, reflecting the project's transition to version 3.x where "shim" terminology is no longer needed.

  • Updated Everything.sln to reference a simplified set of projects (13 projects vs. previous 47)
  • Changed Visual Studio version from 16 to 18 and updated VisualStudioVersion string
  • Removed the SHIM-prefixed relative paths (../) since the solution is now at the repository root

Reviewed changes

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

File Description
SHIM/Microsoft.ApplicationInsights.Shim.sln File deleted as its content is moved to the root Everything.sln
Everything.sln Completely replaced with simplified solution containing core projects, examples, and tests from the former SHIM solution

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

auto-merge was automatically disabled December 18, 2025 22:55

Pull request was closed

@rajkumar-rangaraj rajkumar-rangaraj merged commit f451093 into main Dec 18, 2025
62 checks passed
@rajkumar-rangaraj rajkumar-rangaraj deleted the rajrang/fixEverything.sln branch December 18, 2025 23:06
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.

3 participants