Skip to content

Conversation

@neoscie
Copy link
Contributor

@neoscie neoscie commented Jul 11, 2025

  • Introduces DotNetConsoleLogger for fallback logging.
  • Adds a CI check to validate DI setup using dotnet run -- check-deps to ensure proper dependency resolution and prevent runtime errors.
  • Upgrades dependency packages to their latest versions.

@neoscie neoscie requested a review from Copilot July 11, 2025 08:32
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR introduces the DotNetConsoleLogger static class for fallback logging, adds a CI check to validate dependency injection setup, and upgrades dependency packages to their latest versions.

  • Introduces DotNetConsoleLogger static class for centralized logger initialization and management
  • Adds CI validation using dotnet run -- check-deps to verify DI setup
  • Updates dependency packages (Scrutor, Microsoft.Extensions, test packages) to latest versions

Reviewed Changes

Copilot reviewed 25 out of 29 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
README.md Comprehensive documentation update with installation guide, usage examples, and check-deps explanation
Neolution.DotNet.Console/DotNetConsoleLogger.cs New static logger class providing initialization, access, and shutdown methods
Neolution.DotNet.Console/DotNetConsole.cs Updated to use DotNetConsoleLogger instead of direct NLog setup
Neolution.DotNet.Console/DotNetConsoleException.cs Removed obsolete serialization constructor and attributes
.github/workflows/ci.yml Added CI step to validate DI setup using check-deps command
Various project files Package version updates and demo application restructuring

neoscie and others added 2 commits July 11, 2025 10:33
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@neoscie neoscie merged commit a2a6317 into main Jul 11, 2025
1 check passed
@neoscie neoscie deleted the feature/logger-helper branch July 11, 2025 08:45
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