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

Remove unnecessary interface in service/nodes.go #7013

Merged

Conversation

djaglowski
Copy link
Member

@djaglowski djaglowski commented Jan 24, 2023

This interface was part of #6700 but has been refactored out in the latest implementation.

@djaglowski djaglowski added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Jan 24, 2023
@codecov
Copy link

codecov bot commented Jan 24, 2023

Codecov Report

Base: 90.36% // Head: 90.36% // No change to project coverage 👍

Coverage data is based on head (e51bb01) compared to base (1af31a0).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #7013   +/-   ##
=======================================
  Coverage   90.36%   90.36%           
=======================================
  Files         243      243           
  Lines       14598    14598           
=======================================
  Hits        13191    13191           
  Misses       1138     1138           
  Partials      269      269           
Impacted Files Coverage Δ
service/nodes.go 12.90% <ø> (ø)

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

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

@djaglowski
Copy link
Member Author

This would remove the gonum library, so rather than remove and readd, I'll rebase after #7016 is merged.

@djaglowski djaglowski force-pushed the connectors-nodes-cleanup branch from 45f164a to 65242d8 Compare January 25, 2023 14:01
@djaglowski djaglowski force-pushed the connectors-nodes-cleanup branch from 65242d8 to e51bb01 Compare January 25, 2023 14:02
@djaglowski djaglowski marked this pull request as ready for review January 25, 2023 14:11
@djaglowski djaglowski requested review from a team, codeboten and bogdandrutu January 25, 2023 14:11
@bogdandrutu bogdandrutu merged commit 9e4b354 into open-telemetry:main Jan 25, 2023
@djaglowski djaglowski deleted the connectors-nodes-cleanup branch January 25, 2023 20:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:connector Skip Changelog PRs that do not require a CHANGELOG.md entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants