Skip to content

fix: resolve 71 SonarQube medium maintainability issues in new code#557

Merged
rygel merged 1 commit intodevelopfrom
fix/sonarqube-cleanup
Apr 16, 2026
Merged

fix: resolve 71 SonarQube medium maintainability issues in new code#557
rygel merged 1 commit intodevelopfrom
fix/sonarqube-cleanup

Conversation

@rygel
Copy link
Copy Markdown
Owner

@rygel rygel commented Apr 16, 2026

Summary

  • Fix all 71 medium maintainability issues flagged in new code by SonarQube
  • Auto-format via dotnet format for style issues (SA1122, SA1516, IDE0055, SA1101, IDE0009)
  • Manual fixes for MA0006, MA0002, VSTHRD111, VSTHRD002, VSTHRD105, MA0011

Changes

Rule Count Description
VSTHRD111/MA0004 25 Added ConfigureAwait(false) to test awaits
SA1122 9 Replaced "" with string.Empty
SA1516 14 Added blank lines between elements
MA0006 4 Used string.Equals with StringComparison.Ordinal
MA0002 3 Used StringComparer.Ordinal for dictionaries
SA1101/IDE0009 2 Added this qualification
SA1513 3 Added blank line after closing brace
SA1502 2 Multi-line element declarations
IDE0055 1 Fixed formatting
MA0011 1 Added CultureInfo.InvariantCulture to TryParse
VSTHRD002 1 Replaced .Result with await
VSTHRD105 1 Used TaskScheduler.Default

Test plan

  • Build succeeds with 0 errors
  • All 1487 tests pass (1282 + 159 + 46)

- MA0006: Use string.Equals with StringComparison.Ordinal (4)
- MA0002: Use StringComparer.Ordinal for dictionary lookups (3)
- VSTHRD111/MA0004: Add ConfigureAwait(false) to all test awaits (25)
- VSTHRD002: Replace .Result with await in GitHubAuthServiceTests (1)
- VSTHRD105: Use TaskScheduler.Default in ContinueWith (1)
- MA0011: Add CultureInfo.InvariantCulture to TryParse (1)
- SA1122/SA1516/SA1513/IDE0055/SA1101/IDE0009: Auto-format via dotnet format
- Update sonar-issues.ps1 to support new code period queries
@rygel rygel merged commit f9fcef6 into develop Apr 16, 2026
19 checks passed
@rygel rygel deleted the fix/sonarqube-cleanup branch April 16, 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