-
Notifications
You must be signed in to change notification settings - Fork 6
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
Implement Team Command Parsing #60
Conversation
…to teamcommand
…to teamcommand
Codecov Report
@@ Coverage Diff @@
## master #60 +/- ##
=====================================
Coverage 100% 100%
=====================================
Files 9 10 +1
Lines 302 340 +38
=====================================
+ Hits 302 340 +38
Continue to review full report at Codecov.
|
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.
If I remember, there was also another PR that was merged regarding commands/user.py
. Some of the same issues (static class variables and long initializations that aren't in functions) are present. May want to submit another PR to fix that (and issue, to get our counts up).
Pull Request
Description
Added team command parsing and TeamCommands.md reference. Added tests for parsing.
TODO:
Testing
If testing this change requires extra setup, please document it here:
Ticket(s)
Affects #50, affects #51, affects #52
(Create a copy of that line for each Github Issue affected,
and replace "Affects" with "Closes" if merging this will close the relevant ticket.)