Skip to content

build: remove third-party analyzer packages - #2351

Merged
glennawatson merged 1 commit into
mainfrom
build/remove-third-party-analyzers
Oct 4, 2026
Merged

glennawatson merged 1 commit into
mainfrom
build/remove-third-party-analyzers

Conversation

@glennawatson

Copy link
Copy Markdown
Contributor

Summary

  • Remove the direct SonarAnalyzer dependency and unused analyzer package matchers in Renovate.
  • Keep SonarCloud CI overrides at none. This changes build configuration only, so no new tests are needed.

Why

  • Limit direct analyzer dependencies to the Sharp analyzers, PublicApiSharp, and Microsoft analyzers.

Breaking changes

  • None.

How this was verified

  • Built the full solution in Release and exercised the existing test suite.
  • Checked the restored dependencies and all SonarCloud severity overrides. The hosted SonarCloud scan needs CI.

Notes for the reviewer

  • Start with the shared package references. The Renovate cleanup is mechanical.
  • The SonarCloud workflow stays in place. Its scanner supplies the Sonar analyzers during CI.

Checklist

  • I have read the Contribute guide
  • The PR title follows Conventional Commits
  • Tests cover this change, or the summary says why they do not
  • New or changed public API has XML documentation

- Keep Sharp, PublicApiSharp, and Microsoft analyzers.
- Keep disabled SonarCloud rules for CI scans.
@glennawatson
glennawatson merged commit 970bf3e into main Oct 4, 2026
22 checks passed
@glennawatson
glennawatson deleted the build/remove-third-party-analyzers branch October 4, 2026 08:04
@sonarqubecloud

sonarqubecloud Bot commented Oct 4, 2026

Copy link
Copy Markdown

@codecov

codecov Bot commented Oct 4, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.97%. Comparing base (fbf8edd) to head (ae013c3).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2351   +/-   ##
=======================================
  Coverage   99.97%   99.97%           
=======================================
  Files         219      219           
  Lines       11784    11784           
  Branches     2262     2262           
=======================================
  Hits        11781    11781           
  Misses          3        3           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

1 participant