-
Notifications
You must be signed in to change notification settings - Fork 100
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
fix(extensions): consolidate extensions headers returned to UI by ext… #1473
Conversation
3c1a9f9
to
89745ce
Compare
89745ce
to
a05aefb
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.
LGTM
a05aefb
to
f98a6d6
Compare
…ensions Signed-off-by: Petu Eusebiu <[email protected]>
f98a6d6
to
fc28091
Compare
Codecov Report
@@ Coverage Diff @@
## main #1473 +/- ##
==========================================
+ Coverage 90.95% 90.97% +0.02%
==========================================
Files 103 103
Lines 22608 22605 -3
==========================================
+ Hits 20562 20565 +3
+ Misses 1524 1520 -4
+ Partials 522 520 -2
... and 1 file with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
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.
lgtm
…ensions
What type of PR is this?
Which issue does this PR fix:
What does this PR do / Why do we need it:
If an issue # is not available please add repro steps and logs showing the issue:
Testing done on this change:
Automation added to e2e:
Will this break upgrades or downgrades?
Does this PR introduce any user-facing change?:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.