-
Notifications
You must be signed in to change notification settings - Fork 252
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
ResolveNuGetPackageAssets allocates 3.7% of solution-wide design-time build #6755
Comments
From @davkean on October 4, 2017 23:16 Some of this allocations is due to reading line annotations - turn that off via JsonLoadSettings.LineInfoHandling. |
This is the version of package resolution for non-SDK projects. cc @Pilchie Does this go in project-system or nuget? |
From @Pilchie on February 5, 2018 22:28 project-system has tended to make the changes, but the code lives at https://github.com/NuGet/NuGet.BuildTasks/. |
Yes, I understood as much from #6755 (comment) I also interpreted that as implying I should file the bug where the code lives, which is why I moved it here. |
Moved from: dotnet/NuGet.BuildTasks#81 |
From @davkean on September 26, 2017 12:55
This is literally turning text -> text, can we do this without NewtonSoft.Json? Otherwise, try the lower level reader API that JSON.NET has?
Copied from original issue: dotnet/sdk#1605
The text was updated successfully, but these errors were encountered: