-
-
Notifications
You must be signed in to change notification settings - Fork 444
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
Slack <!channel|desc> & <!channel> support added #856
Conversation
@lanrat: i think i got this for you; would you please give it a try and let me know how it goes? |
Codecov ReportPatch coverage:
📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more Additional details and impacted files@@ Coverage Diff @@
## master #856 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 118 118
Lines 15366 15378 +12
Branches 3153 3155 +2
=========================================
+ Hits 15366 15378 +12
... and 1 file with indirect coverage changes 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 in Codecov by Sentry. |
I'm happy to help test. Is the best way to pull this PR directly? |
Anyway works, as long as you use this branch. Honestly, I'm pretty sure it will work as i even added a test case to verify it. But sometimes for the peace of mind, It's nice hearing from someone else confirming it. It's my understanding that the |
User mentions |
I'm having some difficulty getting a built to test with. I use Docker for my setup. I cloned your branch and built a docker image using the |
No worries, Merging after testing myself. |
Can we get a new release of apprise that includes this? |
Description:
Related issue (if applicable): #849
Support
<!channel>
and<!channel|desc>
directivesChecklist
flake8
)Testing
Anyone can help test this source code as follows: