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 feateuregate.Registry.MustApply, break Apply to return error #5801

Merged
merged 4 commits into from
Aug 4, 2022

Conversation

bogdandrutu
Copy link
Member

Fixes #5656

Signed-off-by: Bogdan bogdandrutu@gmail.com

Signed-off-by: Bogdan <bogdandrutu@gmail.com>
@bogdandrutu bogdandrutu requested review from a team and tigrannajaryan August 3, 2022 16:24
@codecov
Copy link

codecov bot commented Aug 3, 2022

Codecov Report

Merging #5801 (8973f9b) into main (186077d) will decrease coverage by 0.08%.
The diff coverage is 58.33%.

@@            Coverage Diff             @@
##             main    #5801      +/-   ##
==========================================
- Coverage   91.59%   91.50%   -0.09%     
==========================================
  Files         192      192              
  Lines       11473    11472       -1     
==========================================
- Hits        10509    10498      -11     
- Misses        770      778       +8     
- Partials      194      196       +2     
Impacted Files Coverage Δ
service/command.go 78.57% <0.00%> (-9.90%) ⬇️
service/featuregate/gates.go 93.18% <77.77%> (-6.82%) ⬇️
pdata/internal/common.go 94.61% <0.00%> (-0.77%) ⬇️

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

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.

Collector should fail when passed an unregistered feature gate
2 participants