Conversation
There was a problem hiding this comment.
Pull request overview
This PR introduces documentation for agents (automated or human contributors) working on this .NET NuGet package repository. It establishes guidelines for development, testing, review processes, and package publication workflows.
Key Changes:
- Adds agent responsibility guidelines covering development, testing, review, and publication processes
- Provides comprehensive C# development best practices and coding standards
- Defines testing requirements and framework-specific guidance
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| AGENTS.md | Establishes agent responsibilities for development, testing, review, and NuGet package publication workflow |
| .github/agents/CSharpExpert.agent.md | Provides detailed C#/.NET development guidelines including coding standards, async programming patterns, and testing best practices |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
AGENTS.md
Outdated
| 2. Tag the repository with a version following [Semantic Versioning](https://semver.org/). To build a pre-release package, add `-preview` | ||
| 3. The publication workflow will use the tagged commit to generate and publish the NuGet package. |
There was a problem hiding this comment.
The instruction for pre-release packages is incomplete. It should specify where to add '-preview' (e.g., to the version tag like 'v1.0.0-preview') and provide a complete example of a pre-release version tag format.
| 2. Tag the repository with a version following [Semantic Versioning](https://semver.org/). To build a pre-release package, add `-preview` | |
| 3. The publication workflow will use the tagged commit to generate and publish the NuGet package. | |
| 2. Tag the repository with a version following [Semantic Versioning](https://semver.org/). To build a pre-release package, add `-preview` to the version tag (for example, use `v1.0.0-preview` instead of `v1.0.0`). | |
| 3. The publication workflow will use the tagged commit to generate and publish the NuGet package. | |
.github/agents/CSharpExpert.agent.md
Outdated
| bash | ||
| `dotnet tool install -g dotnet-coverage` | ||
| - **Run locally (every time add/modify tests):** | ||
| bash | ||
| `dotnet-coverage collect -f cobertura -o coverage.cobertura.xml dotnet test` |
There was a problem hiding this comment.
The code block language identifier 'bash' on line 157 is not properly formatted as a code fence. It should be formatted as a proper markdown code block with triple backticks.
| bash | |
| `dotnet tool install -g dotnet-coverage` | |
| - **Run locally (every time add/modify tests):** | |
| bash | |
| `dotnet-coverage collect -f cobertura -o coverage.cobertura.xml dotnet test` | |
| ```bash | |
| dotnet tool install -g dotnet-coverage |
- Run locally (every time add/modify tests):
dotnet-coverage collect -f cobertura -o coverage.cobertura.xml dotnet test
.github/agents/CSharpExpert.agent.md
Outdated
| bash | ||
| `dotnet tool install -g dotnet-coverage` | ||
| - **Run locally (every time add/modify tests):** | ||
| bash | ||
| `dotnet-coverage collect -f cobertura -o coverage.cobertura.xml dotnet test` |
There was a problem hiding this comment.
The code block language identifier 'bash' on line 160 is not properly formatted as a code fence. It should be formatted as a proper markdown code block with triple backticks.
| bash | |
| `dotnet tool install -g dotnet-coverage` | |
| - **Run locally (every time add/modify tests):** | |
| bash | |
| `dotnet-coverage collect -f cobertura -o coverage.cobertura.xml dotnet test` | |
| ```bash | |
| dotnet tool install -g dotnet-coverage |
- Run locally (every time add/modify tests):
dotnet-coverage collect -f cobertura -o coverage.cobertura.xml dotnet test
No description provided.