fix(code-quality): resolve SonarCloud code smells and warnings#6
Open
fszymaniak wants to merge 3 commits intomainfrom
Open
fix(code-quality): resolve SonarCloud code smells and warnings#6fszymaniak wants to merge 3 commits intomainfrom
fszymaniak wants to merge 3 commits intomainfrom
Conversation
- S1118: Make Program class static (utility class pattern) - S1192: Extract repeated "Invalid request" string to constant - S6968: Add ProducesResponseType attributes to controller methods - S2386/S3887: Make episode data collections immutable (ReadOnlyCollection) - S5332: Suppress HTTP protocol warning for Railway deployment - S3903: Move Season3Episodes into namespace These changes improve code maintainability, immutability, and API documentation while addressing all non-critical SonarCloud warnings.
- Add .ToList() when assigning OfficeEpisodesData.Episodes to _episodes - Resolves CS0029 compilation error (implicit conversion) - Episode data remains immutable at source, service creates working copy
…ctor - Change Program from static class back to regular class - Add protected constructor to satisfy S1118 SonarCloud rule - Maintains compatibility with WebApplicationFactory<Program> in integration tests - Prevents "static types cannot be used as type arguments" compilation error
fszymaniak
pushed a commit
that referenced
this pull request
Dec 15, 2025
Add detailed analysis and automation scripts for repository branch maintenance: - BRANCH_CLEANUP_REPORT.md: Complete analysis of all branches - Identifies 6 merged branches safe to delete - Lists 14 active PRs needing sync with main - Documents 4 orphaned branches for review - Provides detailed recommendations and next steps - delete-merged-branches.sh: Safe branch deletion script - Verifies branches are fully merged before deletion - Requires user confirmation - Provides detailed progress output - sync-pr-branches.sh: PR synchronization script - Automatically syncs PR branches with main - Handles merge conflicts gracefully - Returns to original branch after completion - BRANCH_MAINTENANCE.md: User guide - Quick start instructions - Script usage documentation - Manual sync procedures - Troubleshooting guide Analysis Summary: - 6 merged branches ready for deletion - 2 Claude PRs need sync (#37, #6) - 12 Dependabot PRs (auto-managed) - 4 orphaned branches need decision
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
These changes improve code maintainability, immutability, and API documentation while addressing all non-critical SonarCloud warnings.