Skip to content

Various fixes to reduce amount of warnings#195

Merged
NSGolova merged 8 commits into
masterfrom
warning-cleanup
May 14, 2026
Merged

Various fixes to reduce amount of warnings#195
NSGolova merged 8 commits into
masterfrom
warning-cleanup

Conversation

@Hermanest

Copy link
Copy Markdown
Member

Updated inspections:

  • MA0046: The delegate must have 2 parameters
  • MA0069: Non-constant static fields should not be visible
  • MA0051: Method is too long (reactive methods cannot be made shorter)
  • MA0011: Use an overload of 'Parse' that has a 'System.IFormatProvider' parameter (redundant boilerplate)
  • MA0099: Use Explicit enum value for '..' instead of num (masks are used frequently so this is not useful)
  • MA0055: Do not use finalizer (destructors are useful)
  • CS0436: Conflicting namespace members from different assemblies (shown even for internal members, doesn't affect anything but produces a warning)

Besides that I've removed some unused code and tried to reduce the amount of nullable mismatches by manually changing annotations.

@Hermanest Hermanest requested a review from NSGolova April 8, 2026 19:46
@NSGolova NSGolova merged commit 8d55d24 into master May 14, 2026
1 check failed
@NSGolova NSGolova deleted the warning-cleanup branch May 14, 2026 14:36
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