-
Notifications
You must be signed in to change notification settings - Fork 293
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
Princess - Chat Commands fix #6433
Conversation
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.
LGTM.
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.
Failing tests need fixed now.
41b292d
to
589ceee
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #6433 +/- ##
=========================================
Coverage 28.42% 28.43%
- Complexity 14093 14095 +2
=========================================
Files 2797 2797
Lines 274761 274773 +12
Branches 48607 48613 +6
=========================================
+ Hits 78113 78119 +6
- Misses 192664 192665 +1
- Partials 3984 3989 +5 ☔ View full report in Codecov by Sentry. |
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.
Looks good!
The chat commands syntax and how they were used in the chat were wrong and did not align with how they actually work.
This not only fix that, as it also fix an error that I commited where I mistakenly believed that you could set the bot behavior using discrete values, this fix also allows that.