Conversation
ptr727
enabled auto-merge (squash)
February 13, 2026 15:01
Contributor
There was a problem hiding this comment.
Pull request overview
This PR represents a major refactoring of the ProjectTemplate project, introducing significant architectural changes and modernizations across all project components.
Changes:
- Upgraded from xUnit v2 to xUnit v3 with AwesomeAssertions, introducing new test fixture patterns (AssemblyFixture, Sequential Test Collection)
- Refactored logging infrastructure from Serilog-only to Microsoft.Extensions.Logging with Serilog bridge, adding configurable LogOptions for library consumers
- Added new CodeGen project for code generation workflows with API integration examples
- Renamed and consolidated GitHub Actions workflows for clarity and consistency
- Comprehensive documentation overhaul with new CODESTYLE.md and AGENTS.md files
- Removed AOT (Ahead-of-Time compilation) support and related dependencies
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| Tests/Tests.csproj | Upgraded to xUnit v3, removed Console project reference, added test-specific .editorconfig suppressions |
| Tests/SampleTest.cs | Simplified test cases, improved naming with scenarios, added SingleInstanceFixture inheritance |
| Tests/LoggingTests.cs | New comprehensive logging tests for LogOptions static API and instance configuration |
| Tests/Fixture.cs | Refactored to use AssemblyFixture and Sequential Test Collection patterns for xUnit v3 |
| Tests/GlobalUsings.cs | Removed unused global usings (System.IO, System.Threading.Tasks) |
| Library/LogOptions.cs | New static logging configuration class with thread-safe factory and logger management |
| Library/Options.cs | New options class for per-instance logger configuration |
| Library/Library.cs | Refactored to use Microsoft.Extensions.Logging with lazy logger initialization |
| Library/Extensions.cs | Converted to use LoggerMessage source generator for structured logging |
| Library/Library.csproj | Removed Serilog dependencies, added Microsoft.Extensions.Logging.Abstractions, moved Husky target |
| Library/GlobalUsings.cs | Switched from Serilog to Microsoft.Extensions.Logging namespaces |
| Library/.editorconfig | Added to suppress CS1591 (missing XML docs) warnings |
| Console/Program.cs | Simplified command execution, integrated Serilog-to-MEL bridge via LoggerFactory |
| Console/CommandLine.cs | Refactored command structure, removed ProcessTask dependencies |
| Console/LoggerFactory.cs | Added Microsoft.Extensions.Logging bridge to Serilog |
| Console/Extensions.cs | Added dual logger extensions for both Serilog and MEL ILogger types |
| Console/Console.csproj | Removed AOT, added Serilog.Extensions.Logging, simplified configuration |
| Console/GlobalUsings.cs | Added Microsoft.Extensions.Logging alongside Serilog |
| Console/.editorconfig | Added to suppress CA1515 (internal types) warning |
| CodeGen/* | New code generation project with API integration, HTTP resilience, and automated code generation |
| Benchmarks/* | Converted #pragma suppressions to attributes, added .editorconfig |
| Docker/Dockerfile | Removed AOT build dependencies (clang, zlib1g-dev) |
| Docker/Build.sh | Removed AOT build targets |
| README.md | Complete restructure with detailed installation, configuration, usage sections |
| CODESTYLE.md | New file extracting coding standards from copilot-instructions |
| AGENTS.md | New AI agent guidance document |
| HISTORY.md | New release history file |
| .editorconfig | Updated modifier order, added ReSharper settings, fixed markdown glob |
| .github/workflows/* | Renamed and reorganized workflows for consistency, added CodeGen workflow |
| .github/copilot-instructions.md | Updated for xUnit v3, AwesomeAssertions, and new project structure |
| ProjectTemplate.slnx | Added CodeGen project, updated build dependencies |
| ProjectTemplate.code-workspace | Added spell-check words, updated launch configurations |
| .gitignore | Added .DS_Store and *.user |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.