Skip to content

Ignore unset notification parameters #519

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.

Sign up for GitHub

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

Merged
d-j-hatton merged 1 commit into from
Mar 19, 2025

Do not try and record notification parameters that are not properly set

cf87ed2
Select commit
Loading
Failed to load commit list.
Merged

Ignore unset notification parameters #519

Do not try and record notification parameters that are not properly set
cf87ed2
Select commit
Loading
Failed to load commit list.
This check has been archived and is scheduled for deletion. Learn more about checks retention
Codecov / codecov/project succeeded Mar 19, 2025 in 0s

28.89% (target 0.00%)

View this Pull Request on Codecov

28.89% (target 0.00%)

Details

Codecov Report

Attention: Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.

Project coverage is 28.89%. Comparing base (2a9f533) to head (cf87ed2).

Additional details and impacted files 
@@            Coverage Diff             @@
##             main     #519      +/-   ##
==========================================
- Coverage   28.90%   28.89%   -0.01%     
==========================================
  Files          79       79              
  Lines       10207    10208       +1     
  Branches     1349     1350       +1     
==========================================
  Hits         2950     2950              
- Misses       7156     7157       +1     
  Partials      101      101
🚀 New features to boost your workflow:
  • ❄ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
View more details on Codecov