Skip to content

Conversation

@voldemort2826
Copy link
Contributor

Reopen for #682 due to branch name change

Address code quality checks introduced in .NET 9. Easiest one like "name/collection initialization can be simplified" are fixed automatically with Visual Studio.

Todo

Level: easy

  • Address CA2201: Exception type System.Exception is not sufficiently specific (commit f084711)
  • Silence/ignore unnecessary warnings
  • Address rest of the warnings before moving on to the <AnalysisMode>Recommended</AnalysisMode>
  • Use ReadyToRun publishing instead of NativeAOT because third-party libraries produced trim warnings
  • Remove custom ChaCha20 implementation in favor of Sodium.Core [require testing with real ChaCha20 stream, but it's rare]

Level: medium

  • Address additional warnings by <AnalysisMode>Recommended</AnalysisMode>

@voldemort2826 voldemort2826 mentioned this pull request Jul 30, 2025
6 tasks
@nilaoda nilaoda force-pushed the main branch 2 times, most recently from d1bea34 to e53faf6 Compare October 27, 2025 14:20
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