Skip to content
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

Log handler #79

Merged
merged 17 commits into from
Jan 4, 2025
Merged

Log handler #79

merged 17 commits into from
Jan 4, 2025

Conversation

d-frey
Copy link
Member

@d-frey d-frey commented Dec 25, 2024

See #78

@d-frey d-frey self-assigned this Dec 25, 2024
Copy link

codecov bot commented Dec 25, 2024

Codecov Report

Attention: Patch coverage is 84.03756% with 34 lines in your changes missing coverage. Please review.

Project coverage is 95.92%. Comparing base (9275b50) to head (5c4ec5f).
Report is 20 commits behind head on main.

Files with missing lines Patch % Lines
src/lib/pq/connection.cpp 64.38% 9 Missing and 17 partials ⚠️
include/tao/pq/poll.hpp 33.33% 8 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #79      +/-   ##
==========================================
- Coverage   96.86%   95.92%   -0.95%     
==========================================
  Files          81       84       +3     
  Lines        2715     2871     +156     
  Branches      164      189      +25     
==========================================
+ Hits         2630     2754     +124     
- Misses         20       37      +17     
- Partials       65       80      +15     

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

@d-frey d-frey marked this pull request as draft December 25, 2024 22:28
@d-frey d-frey marked this pull request as ready for review January 4, 2025 20:41
@d-frey d-frey merged commit 3d7c6fe into main Jan 4, 2025
60 of 62 checks passed
@d-frey d-frey deleted the log branch January 4, 2025 20:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

1 participant