-
Notifications
You must be signed in to change notification settings - Fork 849
Remove orphaned test data files and migrate FSharp.Data to NuGet #19254
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
Co-authored-by: T-Gro <[email protected]>
|
@copilot , pls do: Apparently this test needed the .dll of FSharp.Data. Reproduce locally, fix, verify fix. Failed FSharp.Compiler.Service.Tests.ProjectAnalysisTests.Test Project25 whole project errors [2 s] |
✅ No release notes required |
1 similar comment
✅ No release notes required |
Co-authored-by: T-Gro <[email protected]>
Fixed in commit e7d6071. Changed the test to use FSharp.Data from NuGet package (version 6.6.0) instead of checked-in DLLs. The test now references |
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.