Skip to content

feat: implement sentry logging via slog#601

Merged
NGTmeaty merged 3 commits into
mainfrom
sentry-logging
May 27, 2026
Merged

feat: implement sentry logging via slog#601
NGTmeaty merged 3 commits into
mainfrom
sentry-logging

Conversation

@willmhowes

Copy link
Copy Markdown
Collaborator

No description provided.

@willmhowes willmhowes requested a review from NGTmeaty May 26, 2026 21:32
NGTmeaty
NGTmeaty previously approved these changes May 26, 2026
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 56.61%. Comparing base (54cd94a) to head (fc659b1).

Files with missing lines Patch % Lines
internal/pkg/log/config.go 0.00% 8 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #601      +/-   ##
==========================================
+ Coverage   56.55%   56.61%   +0.05%     
==========================================
  Files         133      133              
  Lines        6751     6760       +9     
==========================================
+ Hits         3818     3827       +9     
- Misses       2554     2556       +2     
+ Partials      379      377       -2     
Flag Coverage Δ
e2etests 41.99% <0.00%> (+0.07%) ⬆️
unittests 29.08% <0.00%> (-0.04%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ 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.

@NGTmeaty NGTmeaty merged commit e991f58 into main May 27, 2026
5 checks passed
@NGTmeaty NGTmeaty deleted the sentry-logging branch May 27, 2026 23:44
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.

3 participants