opentelemetry-collector
opentelemetry-collector copied to clipboard
[configtls] Rename config structs for consistancy
Description: <Describe what has changed.>
Simply renames a few structs in the configtls
package for consistence.
TLSClientSetting
to ClientConfig
TLSServerSetting
to ServerConfig
TLSSetting
to Config
Link to tracking Issue: <Issue number if applicable>
Fixes #9474
I just ran this:
rg "TLSClientSetting" -l --type go | xargs -I{} sh -c "gsed -i 's/TLSClientSetting/ClientConfig/g' {}"
rg "TLSServerSetting" -l --type go | xargs -I{} sh -c "gsed -i 's/TLSServerSetting/ServerConfig/g' {}"
rg "TLSSetting" -l --type go | xargs -I{} sh -c "gsed -i 's/TLSSetting/Config/g' {}"
Will take care of any tests that fail (not expecting anything to fail though).
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 90.89%. Comparing base (
b808e85
) to head (848514f
).
Additional details and impacted files
@@ Coverage Diff @@
## main #9495 +/- ##
=======================================
Coverage 90.89% 90.89%
=======================================
Files 347 347
Lines 18324 18324
=======================================
Hits 16655 16655
Misses 1344 1344
Partials 325 325
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Can this be merged? or am I missing a step?
@arjunmahishi please take a look at the conflict.
@arjunmahishi please take a look at the conflict.
Done