-
Notifications
You must be signed in to change notification settings - Fork 167
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(warehouse): enrich discovery logs #2035
Conversation
✅ Deploy Preview for docs-kargo-akuity-io ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
6d870a7
to
f5f7122
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2035 +/- ##
==========================================
- Coverage 45.79% 45.68% -0.12%
==========================================
Files 236 234 -2
Lines 16077 16042 -35
==========================================
- Hits 7362 7328 -34
+ Misses 8355 8354 -1
Partials 360 360 ☔ View full report in Codecov by Sentry. |
f5f7122
to
601e830
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 after conflicts are resolved
Signed-off-by: Hidde Beydals <hidde@hhh.computer>
Signed-off-by: Hidde Beydals <hidde@hhh.computer>
Signed-off-by: Hidde Beydals <hidde@hhh.computer>
601e830
to
8f4705f
Compare
Related to #1984
This brings the logging for Git and Helm artifact discovery up to the same level as what was already done for container images. Providing much richer data on trace level, and more structured information at the debug level.