Skip to content

fix: resolve code-style issues and suppress AOT warnings#4

Merged
crodriguesbr merged 1 commit into
mainfrom
fix/aot-warnings-servercommand
Dec 4, 2025
Merged

fix: resolve code-style issues and suppress AOT warnings#4
crodriguesbr merged 1 commit into
mainfrom
fix/aot-warnings-servercommand

Conversation

@crodriguesbr
Copy link
Copy Markdown
Contributor

Summary

This PR fixes the code-style CI check failure by addressing formatting issues and suppressing AOT compilation warnings.

Changes

Formatting Fixes

  • Fixed whitespace formatting issues
  • Fixed final newline issues
  • Fixed file encoding (charset) issues
  • Fixed imports ordering

AOT Warnings Suppression

  • Added `UnconditionalSuppressMessage` attributes to `ServerCommand.ExecuteAsync` method
  • Suppresses IL2026 (RequiresUnreferencedCode) and IL3050 (RequiresDynamicCode) warnings
  • These warnings are false positives as we use source-generated JSON serialization (`AppJsonContext`) which is AOT compatible

Documentation

  • Added XML documentation to `ServerCommand` class and `ExecuteAsync` method

Testing

  • `dotnet build --configuration Release`
  • `dotnet format --verify-no-changes`

Related

Fixes the code-style CI check failure on main branch.

- Fix whitespace formatting issues
- Fix final newline issues
- Fix file encoding issues
- Fix imports ordering
- Add UnconditionalSuppressMessage attributes to ServerCommand.ExecuteAsync
  to suppress IL2026/IL3050 AOT warnings for ASP.NET Core Minimal APIs
- Add XML documentation to ServerCommand class and method
@crodriguesbr crodriguesbr merged commit 4578113 into main Dec 4, 2025
2 checks passed
@crodriguesbr crodriguesbr deleted the fix/aot-warnings-servercommand branch December 4, 2025 12:02
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.

1 participant