Remove orphaned test data files and migrate FSharp.Data to NuGet#19254
Merged
Remove orphaned test data files and migrate FSharp.Data to NuGet#19254
Conversation
Co-authored-by: T-Gro <46543583+T-Gro@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Remove orphaned test data files from repository
Remove orphaned test data files from tests/service/data/
Feb 4, 2026
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
github-project-automation
bot
moved this from New
to In Progress
in F# Compiler and Tooling
Copilot stopped work on behalf of
T-Gro due to an error
February 5, 2026 12:19
Contributor
✅ No release notes required |
1 similar comment
Contributor
✅ No release notes required |
Co-authored-by: T-Gro <46543583+T-Gro@users.noreply.github.com>
Copilot
AI
changed the title
Remove orphaned test data files from tests/service/data/
Remove orphaned test data files and migrate FSharp.Data to NuGet
Feb 11, 2026
T-Gro
force-pushed
the
copilot/remove-unused-test-data-files
branch
3 times, most recently
from
8e16576 to
839d76d
Compare
… provider - Revert FSharp.Compiler.Service.Tests.fsproj (remove FSharp.Data PackageReference) - Remove NuGet.Config - Resolve FSharp.Data via #r nuget script resolution at test time - Copy runtime + design-time DLLs to staging dir so type provider can find its design-time component with --simpleresolution - Change Project25 tests to FactSkipOnSignedBuild (skip when NuGet unavailable) - Make Project25.options lazy - Update namespace attribute expectation for NuGet-resolved FSharp.Data
T-Gro
force-pushed
the
copilot/remove-unused-test-data-files
branch
from
839d76d to
10919fd
Compare
abonie
approved these changes
Feb 17, 2026
This was referenced Feb 18, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Removes orphaned test data files and migrates FSharp.Data dependency from checked-in DLLs to NuGet package.
Deleted Files
tests/service/data/sqlite-net-spike/(entire folder){BE87D723-5EAB-4B69-8F65-5EC072EF9E46}appears nowhere elsetests/service/data/FSharp.Data.DesignTime.dlltests/service/data/FSharp.Data.dllFSharp.Data Migration to NuGet
FSharp.Dataversion 6.6.0 as a PackageReference toFSharp.Compiler.Service.Tests.fsprojtests/FSharp.Compiler.Service.Tests/NuGet.Configto enable nuget.org as a package sourceProjectAnalysisTests.fsto reference FSharp.Data.dll from the test assembly output directory usingPathRelativeToTestAssemblyinstead of hardcoded paths to checked-in filesVerification
Code search confirmed zero references:
sqlite-net-spike- only self-references within deleted folderFSharp.Data.DesignTime- no matchesAll tests pass (2,074 passed, 0 failed, 29 skipped).
Original prompt
This pull request was created from Copilot chat.
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.