-
Notifications
You must be signed in to change notification settings - Fork 118
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
chore: remove decoder injector interface from webhook #1563
chore: remove decoder injector interface from webhook #1563
Conversation
Signed-off-by: geoffrey1330 <israelgeoffrey13@gmail.com>
✅ Deploy Preview for keptn-lifecycle-toolkit ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #1563 +/- ##
==========================================
- Coverage 63.22% 63.12% -0.10%
==========================================
Files 142 142
Lines 10794 10800 +6
==========================================
- Hits 6824 6817 -7
- Misses 3715 3724 +9
- Partials 255 259 +4
... and 10 files with indirect coverage changes
Flags with carried forward coverage won't be shown. Click here to find out more. |
Signed-off-by: geoffrey1330 <israelgeoffrey13@gmail.com>
Signed-off-by: geoffrey1330 <israelgeoffrey13@gmail.com>
Signed-off-by: geoffrey1330 <israelgeoffrey13@gmail.com>
Signed-off-by: geoffrey1330 <israelgeoffrey13@gmail.com>
Signed-off-by: geoffrey1330 <israelgeoffrey13@gmail.com>
Kudos, SonarCloud Quality Gate passed!
|
Thank you @geoffrey1330 for your contribution! :) |
Fixes #1445 Removed Decoder Injector interface from webhook and explicitly initialized the decoder in the operator main
Signed-off-by: geoffrey1330 israelgeoffrey13@gmail.com