-
Notifications
You must be signed in to change notification settings - Fork 691
Add support for statuspage.io #1120
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
Conversation
And also fix parameters in docstings
As I see you're almost ready :) |
Looks great! |
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 #1120 +/- ##
==========================================
- Coverage 35.95% 34.09% -1.87%
==========================================
Files 39 40 +1
Lines 7158 7588 +430
Branches 1032 1052 +20
==========================================
+ Hits 2574 2587 +13
- Misses 4474 4891 +417
Partials 110 110
... and 8 files 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. |
This way it feels better
In general, I think this is more useful. Yet there is a reason to use PUT sometimes, but when? And how do we call functions with it?
Thank you for your work @hugmouse ! |
Working on statuspage.io wrapper here!
Upon merge close issue #1026.