Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deprecate last funcs/structs in componenthelper #5069

Merged
merged 2 commits into from
Mar 25, 2022

Conversation

bogdandrutu
Copy link
Member

Fixes #4681

These funcs/structs are not used, and in case we need them in multiple places we can add them to component directly.

Signed-off-by: Bogdan Drutu [email protected]

@bogdandrutu bogdandrutu requested review from a team and tigrannajaryan March 23, 2022 13:31
@codecov
Copy link

codecov bot commented Mar 23, 2022

Codecov Report

Merging #5069 (3ec8c6d) into main (22d89f0) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #5069      +/-   ##
==========================================
- Coverage   89.51%   89.50%   -0.01%     
==========================================
  Files         183      183              
  Lines       11040    11041       +1     
==========================================
  Hits         9882     9882              
- Misses        936      937       +1     
  Partials      222      222              
Impacted Files Coverage Δ
component/componenthelper/component.go 100.00% <ø> (ø)
internal/testcomponents/example_extension.go 70.00% <0.00%> (-7.78%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 22d89f0...3ec8c6d. Read the comment docs.

Fixes open-telemetry#4681

These funcs/structs are not used, and in case we need them in multiple places we can add them to `component` directly.

Signed-off-by: Bogdan Drutu <[email protected]>
@bogdandrutu bogdandrutu changed the title Deprecate last funcs/structs in componenthelper. Deprecate last funcs/structs in componenthelper Mar 23, 2022
@bogdandrutu bogdandrutu merged commit 6f4057b into open-telemetry:main Mar 25, 2022
@bogdandrutu bogdandrutu deleted the rmcomphelper branch March 25, 2022 08:58
Nicholaswang pushed a commit to Nicholaswang/opentelemetry-collector that referenced this pull request Jun 7, 2022
Fixes open-telemetry#4681

These funcs/structs are not used, and in case we need them in multiple places we can add them to `component` directly.

Signed-off-by: Bogdan Drutu <[email protected]>
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.

[Proposal] Try to move helpers to create [*]Factory types in component directly.
2 participants