Skip to content

Conversation

@fuskovic
Copy link
Member

@fuskovic fuskovic commented Oct 29, 2025

Closes: #4486

Signed-off-by: fuskovic <[email protected]>
@netlify
Copy link

netlify bot commented Oct 29, 2025

Deploy Preview for docs-kargo-io ready!

Name Link
🔨 Latest commit 097f65b
🔍 Latest deploy log https://app.netlify.com/projects/docs-kargo-io/deploys/691237006b84090008065fe9
😎 Deploy Preview https://deploy-preview-5305.docs.kargo.io
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@fuskovic fuskovic self-assigned this Oct 31, 2025
@fuskovic fuskovic added priority/high Needs to be addressed sooner rather than later area/external-webhooks Affects the server that responds to webhooks originating from external systems kind/enhancement An entirely new feature labels Oct 31, 2025
@codecov
Copy link

codecov bot commented Nov 4, 2025

Codecov Report

❌ Patch coverage is 76.62835% with 61 lines in your changes missing coverage. Please review.
✅ Project coverage is 56.12%. Comparing base (60fba34) to head (097f65b).
⚠️ Report is 30 commits behind head on main.

Files with missing lines Patch % Lines
pkg/webhook/kubernetes/external/validation.go 0.00% 33 Missing ⚠️
pkg/webhook/external/generic.go 76.69% 23 Missing and 1 partial ⚠️
pkg/webhook/external/refresh.go 90.24% 3 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5305      +/-   ##
==========================================
+ Coverage   55.98%   56.12%   +0.14%     
==========================================
  Files         405      409       +4     
  Lines       29858    30189     +331     
==========================================
+ Hits        16715    16945     +230     
- Misses      12177    12271      +94     
- Partials      966      973       +7     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@fuskovic fuskovic requested review from hiddeco and krancour November 10, 2025 19:01
@fuskovic fuskovic marked this pull request as ready for review November 10, 2025 19:01
@fuskovic fuskovic requested review from a team as code owners November 10, 2025 19:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/external-webhooks Affects the server that responds to webhooks originating from external systems kind/enhancement An entirely new feature priority/high Needs to be addressed sooner rather than later

Projects

None yet

Development

Successfully merging this pull request may close these issues.

generic webhook receiver

4 participants