django-wiki / django-nyt

Notification system for Django with batteries included: Email digests, user settings, JSON API
Apache License 2.0
144 stars 47 forks source link

Change the return value of `notify()` (as already mentioned in documentation) #134

Closed benjaoming closed 10 months ago

benjaoming commented 10 months ago
codecov[bot] commented 10 months ago

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (08571fc) 74.87% compared to head (91e3fcc) 76.73%.

Files Patch % Lines
django_nyt/utils.py 85.71% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #134 +/- ## ========================================== + Coverage 74.87% 76.73% +1.85% ========================================== Files 23 23 Lines 621 619 -2 ========================================== + Hits 465 475 +10 + Misses 156 144 -12 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

benjaoming commented 10 months ago

This is moving along nicely, making a pre-release after merging this in order to try it out :+1:

benjaoming commented 10 months ago

Any input well-received!