Marc Paine
Marc Paine
@dotnet/dotnet-analyzers
Triage: @jlaanstra @Sergio0694 were ya'll going to put up a PR per the suggestion above or looking for someone else to?
That is super tricky to test and no testing is really sufficient here. @dsplaisted any suggestions on how to be safe about moving around targets?
@copilot I only want to impact the results from dotnet workload search version. Changing nugetpackagedownloader is too central and may have potential side effects for other commands that also use...
Tested and reviewed. The copilot tried to run format on the files changed, got confused, and formatted the whole project. The full actual changes are in two files. src/Cli/dotnet/Commands/Workload/Search/WorkloadSearchVersionsCommand.cs test/dotnet.Tests/CommandTests/Workload/Search/GivenDotnetWorkloadSearch.cs...
@copilot can you fix the merge conflicts?
@nagilson Got copilot to fix the merge conflicts. I tested this a while ago but never got a review. Most of the changes are formatting as this was before we...
Tested in codespaces. Was able to specify different versions when running restore and confirmed it restored the right versions.
Tested in codespaces. The original repro only created one solution folder and I was able to build the project.
@tmat I believe you know source link well so may know if this is the right solution here. I just had copilot follow what the original issue stated but wanted...