Skip to content

Enforce AGENTS.md conventions: TreatWarningsAsErrors and MSTest parallelization#68

Merged
Malcolmnixon merged 2 commits intomainfrom
copilot/analyze-agents-md-violations
Dec 10, 2025
Merged

Enforce AGENTS.md conventions: TreatWarningsAsErrors and MSTest parallelization#68
Malcolmnixon merged 2 commits intomainfrom
copilot/analyze-agents-md-violations

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Dec 10, 2025

AGENTS.md specifies <WarningsAsErrors>true</WarningsAsErrors> but the project files were missing this setting, and MSTest analyzer warnings (MSTEST0001) were present in builds.

Changes

  • Project configuration: Added <TreatWarningsAsErrors>true</TreatWarningsAsErrors> to both library and test project files
  • Test parallelization: Created AssemblyInfo.cs with [assembly: Parallelize(Workers = 0, Scope = ExecutionScope.MethodLevel)] to explicitly configure MSTest parallelization and eliminate MSTEST0001 warnings

Build now produces zero warnings with warnings-as-errors enforcement active.

Original prompt

Please analyze the project using the new AGENTS.md file and identify any violations of its conventions and guidelines.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

…est parallelization

Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
Copilot AI changed the title [WIP] Analyze project for AGENTS.md convention violations Enforce AGENTS.md conventions: TreatWarningsAsErrors and MSTest parallelization Dec 10, 2025
Copilot AI requested a review from Malcolmnixon December 10, 2025 19:27
Copy link
Copy Markdown
Member

@Malcolmnixon Malcolmnixon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

@Malcolmnixon Malcolmnixon marked this pull request as ready for review December 10, 2025 19:33
@Malcolmnixon Malcolmnixon merged commit 7456a52 into main Dec 10, 2025
8 checks passed
@Malcolmnixon Malcolmnixon deleted the copilot/analyze-agents-md-violations branch December 10, 2025 19:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants