Skip to content

Remove unused private fields from SiteConfig.cs#763

Merged
poppastring merged 2 commits into
mainfrom
copilot/remove-unused-private-fields
Jan 17, 2026
Merged

Remove unused private fields from SiteConfig.cs#763
poppastring merged 2 commits into
mainfrom
copilot/remove-unused-private-fields

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Jan 17, 2026

The SiteConfig class contained unused private fields _cdnFrom and _cdnTo that were never referenced, causing CS0169 compiler warnings.

Changes

  • Removed private string _cdnFrom; and private string _cdnTo; declarations
  • Public auto-properties CdnFrom and CdnTo remain unchanged and functional

The private fields were declared but never used anywhere in the codebase—the public properties handle all CDN configuration throughout the application (FileSystemBinaryManager, admin UI, configuration serialization).

Original prompt

This section details on the original issue you should resolve

<issue_title>Remove unused private fields in SiteConfig.cs</issue_title>
<issue_description>The file DasBlog.Services\ConfigFile\SiteConfig.cs contains unused private fields (_cdnFrom and _cdnTo). These fields are never referenced and can be safely removed to clean up the code and resolve CS0169 warnings. Only the public properties CdnFrom and CdnTo are used.

Suggested fix:

  • Remove the private fields _cdnFrom and _cdnTo from the SiteConfig class.

This will improve code clarity and eliminate unnecessary compiler warnings.</issue_description>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: poppastring <8739371+poppastring@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove unused private fields in SiteConfig.cs Remove unused private fields from SiteConfig.cs Jan 17, 2026
Copilot AI requested a review from poppastring January 17, 2026 19:17
@poppastring poppastring marked this pull request as ready for review January 17, 2026 19:22
@poppastring poppastring merged commit 2a37128 into main Jan 17, 2026
3 checks passed
@poppastring poppastring deleted the copilot/remove-unused-private-fields branch January 17, 2026 19:23
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.

Remove unused private fields in SiteConfig.cs

2 participants