Skip to content

[Infra] Update banned API analyzers - #4068

Closed
martincostello wants to merge 6 commits into
open-telemetry:mainfrom
martincostello:banned-api-analyzers
Closed

[Infra] Update banned API analyzers#4068
martincostello wants to merge 6 commits into
open-telemetry:mainfrom
martincostello:banned-api-analyzers

Conversation

@martincostello

@martincostello martincostello commented Apr 7, 2026

Copy link
Copy Markdown
Member

Relates to #4064.

Changes

Ban constructors for Activity and Meter that do not support setting the telemetry schema URL.

Merge requirement checklist

  • CONTRIBUTING guidelines followed (license requirements, nullable enabled, static analysis, etc.)
  • Unit tests added/updated
  • Appropriate CHANGELOG.md files updated for non-trivial changes
  • Changes in public API reviewed (if applicable)

@github-actions github-actions Bot added infra Infra work - CI/CD, code coverage, linters dependencies Pull requests that update a dependency file labels Apr 7, 2026
@codecov

codecov Bot commented Apr 7, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.85%. Comparing base (435cf5f) to head (e8555fb).
✅ All tests successful. No failed tests found.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #4068      +/-   ##
==========================================
+ Coverage   75.76%   75.85%   +0.08%     
==========================================
  Files         438      438              
  Lines       18002    18002              
==========================================
+ Hits        13639    13655      +16     
+ Misses       4363     4347      -16     
Flag Coverage Δ
unittests-Contrib.Shared.Tests 89.79% <ø> (ø)
unittests-Exporter.Geneva 54.93% <ø> (+0.21%) ⬆️
unittests-Exporter.InfluxDB 95.75% <ø> (ø)
unittests-Exporter.OneCollector 94.67% <ø> (ø)
unittests-Extensions 90.61% <ø> (ø)
unittests-Extensions.Enrichment 100.00% <ø> (ø)
unittests-Extensions.Enrichment.AspNetCore 86.27% <ø> (ø)
unittests-Extensions.Enrichment.Http 94.33% <ø> (ø)
unittests-Instrumentation.AWS 85.08% <ø> (ø)
unittests-Instrumentation.AspNet 77.50% <ø> (ø)
unittests-Instrumentation.AspNetCore 87.34% <ø> (ø)
unittests-Instrumentation.EntityFrameworkCore 81.02% <ø> (ø)
unittests-Instrumentation.GrpcCore 91.18% <ø> (ø)
unittests-Instrumentation.GrpcNetClient 78.50% <ø> (ø)
unittests-Instrumentation.Http 74.81% <ø> (ø)
unittests-Instrumentation.Owin 88.88% <ø> (ø)
unittests-Instrumentation.Process 100.00% <ø> (ø)
unittests-Instrumentation.Remoting 65.59% <ø> (ø)
unittests-Instrumentation.Runtime 100.00% <ø> (ø)
unittests-Instrumentation.SqlClient 82.68% <ø> (ø)
unittests-Instrumentation.StackExchangeRedis 94.02% <ø> (ø)
unittests-OpAmp.Client 85.47% <ø> (ø)
unittests-PersistentStorage 69.20% <ø> (+1.46%) ⬆️
unittests-Resources.AWS 73.61% <ø> (ø)
unittests-Resources.Azure 87.67% <ø> (ø)
unittests-Resources.Container 67.34% <ø> (ø)
unittests-Resources.Gcp 71.42% <ø> (ø)
unittests-Resources.Host 72.26% <ø> (ø)
unittests-Resources.OperatingSystem 76.98% <ø> (ø)
unittests-Resources.Process 90.47% <ø> (ø)
unittests-Resources.ProcessRuntime 79.59% <ø> (ø)
unittests-Sampler.AWS 95.97% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 4 files with indirect coverage changes

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Comment thread build/BannedSymbols.txt Outdated
@martincostello martincostello added the keep-open Prevents issues and pull requests being closed as stale label Apr 12, 2026
@martincostello martincostello changed the title [Infra] Add banned API analyzers [Infra] Update banned API analyzers Apr 16, 2026
Ban constructors for `Activity` and `Meter` that do not support setting the telemetry schema URL.

Relates to open-telemetry#4064.
@github-actions github-actions Bot removed the dependencies Pull requests that update a dependency file label Apr 20, 2026
Fix signatures by removing spaces between parameters.
@martincostello

Copy link
Copy Markdown
Member Author

I'm going to close this for now rather than leave it lying around indefinitely until all the providers listed in #4064 are completed. This can be reopened once that issue is closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

infra Infra work - CI/CD, code coverage, linters keep-open Prevents issues and pull requests being closed as stale

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant