Remove unused member from confmap.Resolver

[bogdandrutu]

This may be a breaking change, but should not be really important...

[Aneurysm9]

This may be a breaking change, but should not be really important...

I don't think we get to make that decision for users of this framework.

[bogdandrutu]

@Aneurysm9 who gets?

[bogdandrutu]

Extracting @Aneurysm9 comment from the Slack channel

I think that means it should be included in the changelog so that users who are trying to figure out what broke when they update aren’t stumped by missing changes in the changelog

I think it is a fair comment, doing that now.

[codecov]

Codecov Report

Base: 91.65% // Head: 91.63% // Decreases project coverage by -0.01% ⚠️

Coverage data is based on head (525a6a2) compared to base (3899d3e).
Patch has no changes to coverable lines.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #6489      +/-   ##
==========================================
- Coverage   91.65%   91.63%   -0.02%     
==========================================
  Files         240      240              
  Lines       13817    13826       +9     
==========================================
+ Hits        12664    12670       +6     
- Misses        923      925       +2     
- Partials      230      231       +1     

Impacted Files	Coverage Δ
confmap/resolver.go	96.34% <ø> (ø)
service/telemetry/telemetry.go	80.00% <0.00%> (-3.34%)	⬇️
service/unmarshaler.go	100.00% <0.00%> (ø)
service/telemetry/config.go	100.00% <0.00%> (ø)

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.