-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Move config.Map to its own package which does not depend on any component concept #5237
Conversation
b4f5546
to
7fc2c8c
Compare
Codecov Report
@@ Coverage Diff @@
## main #5237 +/- ##
==========================================
+ Coverage 90.88% 90.97% +0.08%
==========================================
Files 190 191 +1
Lines 11424 11392 -32
==========================================
- Hits 10383 10364 -19
+ Misses 820 809 -11
+ Partials 221 219 -2
Continue to review full report at Codecov.
|
bf0717e
to
3a5ebd4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't have good ideas for the name but I am in favor of moving this to its own package; I think it will allow for gradually marking parts of the codebase as stable
This PR was marked stale due to lack of activity. It will be closed in 14 days. |
More than that, I think this package becomes completely independent on any other collector component/concept. |
@mx-psi decided on |
c902592
to
71a7c39
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a few minor comments/suggestion and a question.
2bba583
to
61b1f56
Compare
@codeboten @mx-psi all updates done, using |
001e72d
to
b8734b5
Compare
c9392f5
to
b947482
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Please resolve conflicts |
…nent concept Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
Followup after open-telemetry#5237 when I forgot to change the name of this func accordingly to the new Conf name. Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
Followup after open-telemetry#5237 when I forgot to change the name of this func accordingly to the new Conf name. Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
Followup after open-telemetry#5237 when I forgot to change the name of this func accordingly to the new Conf name. Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
Followup after open-telemetry#5237 when I forgot to change the name of this func accordingly to the new Conf name. Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
Followup after open-telemetry#5237 when I forgot to change the name of this func accordingly to the new Conf name. Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
Signed-off-by: Bogdan Drutu bogdandrutu@gmail.com