mirror of
https://github.com/okieselbach/AutopilotMonitor.git
synced 2026-10-22 19:55:35 +00:00
Oliver Kieselbach
98d87a8579
The root .editorconfig raises IDE0051, IDE0052 and IDE0060 (non-public methods) to warning, and Directory.Build.props sets EnforceCodeStyleInBuild, so the warning freeze turns them into build errors in every project, the net48 agent included. Cleanup to start at zero, each finding checked for leftover vs. missing behaviour (none was a feature that silently never ran): - 38 injected dependencies nobody read: loggers, configuration, and services whose last read had moved to a live replacement (the IME reseed writes through the rule repository, the offboarding poison OpsEvent comes from the handler, the vulnerability gate sits in the correlate handler). Removed with their constructor parameters and the argument at every construction site. - unused private members: SelfUpdater.IsNewerVersion, a GatherRuleExecutor wrapper, test constants and an unused test overload. RjPhaseCompletedFirstDeployment now replaces the two literal 110s it names. - unused parameters removed together with their arguments; test fakes bound as method groups use discards. - the commented-out backend-URL connectivity check and its plumbing are gone; the reason the backend stays out remains as a comment. - DiagnosticsPackageService names the blob in its upload-failure warnings, as its doc already said. - stale docs fixed on the way: OnboardedAt no longer claims a maintenance backfill (shared manifest and wire types regenerated), the vulnerability functions' policy comments match the catalog, a misplaced summary moved to its method. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
12 lines
586 B
INI
12 lines
586 B
INI
# Analyzer severities for the .NET build. Directory.Build.props runs the IDE rules in
|
|
# `dotnet build` (EnforceCodeStyleInBuild), and the warning freeze makes every warning an error.
|
|
root = true
|
|
|
|
[*.cs]
|
|
# Unused private members, private members that are written but never read, and unused parameters
|
|
# of non-public methods. An injected dependency nobody reads is a feature that silently never runs.
|
|
dotnet_diagnostic.IDE0051.severity = warning
|
|
dotnet_diagnostic.IDE0052.severity = warning
|
|
dotnet_diagnostic.IDE0060.severity = warning
|
|
dotnet_code_quality_unused_parameters = non_public
|