Closed nekowinston closed 1 year ago
We should add a deprecation warning to growlnotify before closing #875. However, that doesn't necessarily need to be done in this pull request.
Thank you for the review comments, I'll update the PR accordingly.
@srl295 I don't have a mac to test this on. Can you test it out?
(Note that I will eventually merge this even if nobody tests it.)
Here's a screenshot of the notifications for the current implementation, after the reviews.
Thank you both for your work on this!
Would close #875.
pytest
failed for me locally, even before adding the feature, unsure why, exactly.