Skip to content

fix: Validate message fields before protobuf encoding for better error messages#333

Merged
BulkBeing merged 7 commits into
mainfrom
validate-message-fields
Mar 12, 2026
Merged

fix: Validate message fields before protobuf encoding for better error messages#333
BulkBeing merged 7 commits into
mainfrom
validate-message-fields

more tests

a471525
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Mar 12, 2026 in 0s

94.43% (+0.33%) compared to b37f25f

View this Pull Request on Codecov

94.43% (+0.33%) compared to b37f25f

Details

Codecov Report

❌ Patch coverage is 94.11765% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 94.43%. Comparing base (b37f25f) to head (a471525).

Files with missing lines Patch % Lines
...pynumaflow/reducestreamer/servicer/task_manager.py 83.33% 2 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #333      +/-   ##
==========================================
+ Coverage   94.10%   94.43%   +0.33%     
==========================================
  Files          66       67       +1     
  Lines        3137     3182      +45     
  Branches      168      179      +11     
==========================================
+ Hits         2952     3005      +53     
+ Misses        151      142       -9     
- Partials       34       35       +1     

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