-
Notifications
You must be signed in to change notification settings - Fork 8
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
Error message improvements #321
base: master
Are you sure you want to change the base?
Conversation
This reverts commit 156b9c3.
This reverts commit 1f64b6a.
Should fix current CI issues
…tUK/KoalaBot into ErrorMessageImprovements
Codecov Report
@@ Coverage Diff @@
## master #321 +/- ##
==========================================
- Coverage 86.72% 86.69% -0.04%
==========================================
Files 111 111
Lines 8161 8172 +11
==========================================
+ Hits 7078 7085 +7
- Misses 1083 1087 +4
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
unit tests need to be added to allow this to pass CodeCov check
Improvements based on pull request response. As CheckFailure is raised due to a commands.Check() error, it is unclear specifically what can cause it.
…tUK/KoalaBot into ErrorMessageImprovements
# Conflicts: # koalabot.py
Summary
Improved error messages for CheckFailure, Forbidden, MissingRequiredArgument, and cog specific errors in verify and RFR
close #84
Checklist
CHANGELOG.md
under the[Unreleased]
heading?documentation.json
?