-
Notifications
You must be signed in to change notification settings - Fork 860
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
preparations for soft deprecation of legacy notification args #1377
Conversation
removes all code related to log levels and title, since that is not used anyway this also gets rid of slackrus dependency
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #1377 +/- ##
=======================================
Coverage ? 64.76%
=======================================
Files ? 24
Lines ? 2310
Branches ? 0
=======================================
Hits ? 1496
Misses ? 718
Partials ? 96
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
this PR contains initial work for an eventual soft deprecation of legacy notification args.
the main feature added is the
notify-upgrade
command that can be used as an easy upgrade path for users that currently rely on the legacy notification wrappers.