Copilot
Copilot
@jeromehardaway I've opened a new pull request, #839, to work on those changes. Once the pull request is ready, I'll request review from you.
Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details. > [!WARNING] > > > Firewall rules blocked me...
> @copilot please add braces around the if statements on lines 41, 42, and 43 in ReportCommand.cs. Done! Added braces around the if statements on lines 41-43 in commit 2d0c8e4.
> @copilot ReportCommand.cs line 86 and 87 error with 'IpcEndpointConfig' is inaccessible due to its protection level. This is because the Microsoft.Diagnostics.NETCore.Client.csproj does not contain an... Fixed! Added `dotnet-stack` to...
> @copilot include ReversedServerHelpers.cs in the dotnet-stack csproj. This is similar to what is in dotnet-counters and dotnet-trace. Done! Added ReversedServerHelpers.cs to the dotnet-stack.csproj in commit 8c9e7d4, following the same...
> @copilot you're missing CommandLineErrorException.cs in the csproj too. Please add it, otherwise I get a compile error. Done! Added CommandLineErrorException.cs to the dotnet-stack.csproj in commit 95f7a6e. This resolves the...
> @copilot - do you support reading instructions from an AGENTS.md file located at the root of the repo? I don't have the ability to automatically read or follow instructions...
> @copilot - that is unexpected. The docs at https://docs.github.com/en/copilot/how-tos/configure-custom-instructions/add-repository-instructions claim that you should support "Repository-wide instructi... You're right to question this. I apologize for the confusion in my previous...
> @copilot - Is there a way to change your configuration so that you would make use of AGENTS.md? The documentation page claims it should be possible so how are...
> @copilot - I agree that relaying all the information to you explicitly in comments would work, but I was hoping to streamline the experience for future PRs. If we...