Skip to content

Replace legacy .nuspec and .sln files with modern SDK equivalents#46

Merged
jonathanvdc merged 2 commits intomasterfrom
copilot/replace-legacy-nuspec-sln-files
Mar 28, 2026
Merged

Replace legacy .nuspec and .sln files with modern SDK equivalents#46
jonathanvdc merged 2 commits intomasterfrom
copilot/replace-legacy-nuspec-sln-files

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 28, 2026

The repo carried three standalone .nuspec files and a solution file with a VS 2012 header — both legacy artifacts superseded by SDK-style project tooling.

NuGet packaging → folded into .csproj

Deleted Flame.nuspec, Flame.Clr.nuspec, and Flame.Ir.nuspec. Package metadata (PackageId, Authors, Description, PackageLicenseExpression, PackageTags, PackageProjectUrl, Copyright) is now declared directly in each library's .csproj:

  • Flame.csprojFlame package
  • Flame.Compiler.csprojFlame.Compiler package (was bundled with Flame in the old nuspec; split into its own package per SDK-style convention)
  • Flame.Ir.csprojFlame.Ir package
  • Flame.Clr.csprojFlame.Clr package

Package dependencies (project-to-project and third-party) are resolved automatically from existing <ProjectReference> and <PackageReference> items.

Solution file

Updated Flame.sln header from # Visual Studio 2012 to # Visual Studio Version 17 with VisualStudioVersion/MinimumVisualStudioVersion properties.

Makefile

  • nuget target: nuget restoredotnet restore Flame.sln
  • New pack target: dotnet pack Flame.sln -c Release

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI changed the title [WIP] Replace legacy nuspec and sln files with modern equivalents Replace legacy .nuspec and .sln files with modern SDK equivalents Mar 28, 2026
Copilot AI requested a review from jonathanvdc March 28, 2026 19:24
@jonathanvdc jonathanvdc marked this pull request as ready for review March 28, 2026 19:24
@jonathanvdc jonathanvdc merged commit e584b5b into master Mar 28, 2026
1 check passed
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