Deprecate config.[Receiver|Processor|Exporter|Extension].ID() and SetName()

Open bogdandrutu opened this issue 3 years ago • 2 comments

trafficstars

Updates https://github.com/open-telemetry/opentelemetry-collector/issues/4714

Aug 08 '22 23:08 bogdandrutu

Codecov Report

Base: 91.21% // Head: 91.20% // Decreases project coverage by -0.00% :warning:

Coverage data is based on head (2ce6510) compared to base (4ff1ff3). Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #5870      +/-   ##
==========================================
- Coverage   91.21%   91.20%   -0.01%     
==========================================
  Files         245      245              
  Lines       14194    14201       +7     
==========================================
+ Hits        12947    12952       +5     
- Misses        997      999       +2     
  Partials      250      250

Impacted Files	Coverage Δ
component/config.go	`87.50% <ø> (ø)`
component/exporter.go	`100.00% <ø> (ø)`
component/extension.go	`100.00% <ø> (ø)`
component/identifiable.go	`100.00% <ø> (ø)`
component/processor.go	`100.00% <ø> (ø)`
component/receiver.go	`100.00% <ø> (ø)`
config/settings.go	`66.66% <ø> (-33.34%)`	:arrow_down:
exporter/exporterhelper/common.go	`90.80% <100.00%> (ø)`
exporter/exporterhelper/logs.go	`80.64% <100.00%> (ø)`
exporter/exporterhelper/metrics.go	`80.64% <100.00%> (ø)`
... and 21 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.