Skip to content

Conversation

xperiandri
Copy link
Collaborator

No description provided.

@xperiandri xperiandri requested a review from Copilot August 17, 2025 20:27
Copy link

@Copilot 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 migrates all Visual Studio solution files from the legacy .sln format to the newer .slnx format. The migration includes updating MIME type mappings to support the new file extensions and updating build scripts to reference the new solution files.

  • Migration of main solution files from .sln to .slnx format
  • Addition of MIME type mappings for .slnf (JSON) and .slnx (XML) file extensions
  • Update to build script to reference the new solution file format

Reviewed Changes

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

Show a summary per file
File Description
src/FSharp.Data.GraphQL.Client/MimeTypes.fs Added MIME type mappings for new solution file formats
samples/star-wars-fabulous-client/StarWars.slnx New XML-based solution file replacing the old .sln
samples/star-wars-fabulous-client/StarWars.sln Removal of legacy solution file
build/Program.fs Updated build script to reference .slnx file
FSharp.Data.GraphQL.slnx New main solution file in XML format
FSharp.Data.GraphQL.sln Removal of legacy main solution file
FSharp.Data.GraphQL.Integration.slnx New integration solution file
FSharp.Data.GraphQL.Integration.sln Removal of legacy integration solution file
FSharp.Data.GraphQL.Build.slnx New build solution file
FSharp.Data.GraphQL.Build.sln Removal of legacy build solution file

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@xperiandri xperiandri merged commit adea02b into dev Aug 17, 2025
3 checks passed
@xperiandri xperiandri deleted the slnx branch August 17, 2025 22:21
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