-
-
Notifications
You must be signed in to change notification settings - Fork 8
Stat[u]s Command #130
Stat[u]s Command #130
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.
You've accidentally included changes from your other PR, try and make a branch for next time
src/main/java/org/cascadebot/cascadebot/commands/informational/StatusCommand.java
Outdated
Show resolved
Hide resolved
src/main/java/org/cascadebot/cascadebot/commands/informational/StatusCommand.java
Outdated
Show resolved
Hide resolved
src/main/java/org/cascadebot/cascadebot/commands/informational/StatusCommand.java
Outdated
Show resolved
Hide resolved
Oops, sorry |
src/main/java/org/cascadebot/cascadebot/commands/core/StatusCommand.java
Outdated
Show resolved
Hide resolved
src/main/java/org/cascadebot/cascadebot/commands/core/StatusCommand.java
Outdated
Show resolved
Hide resolved
src/main/java/org/cascadebot/cascadebot/commands/core/StatusCommand.java
Outdated
Show resolved
Hide resolved
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.
src/main/java/org/cascadebot/cascadebot/commands/core/StatusCommand.java
Outdated
Show resolved
Hide resolved
@binaryoverload #128 has been merged. |
Meeting discussion around this PRThis command is currently closest to a "stats" command so it should be changed from
|
src/main/java/org/cascadebot/cascadebot/commands/core/StatsCommand.java
Outdated
Show resolved
Hide resolved
src/main/java/org/cascadebot/cascadebot/commands/core/StatsCommand.java
Outdated
Show resolved
Hide resolved
@binaryoverload @weeryan17 did all the stuff that cascade told me to |
Pull request
Added/Changed
Added Status Command
I have made this a draft PR because I don't know if y'all even want this command, and I need feedback on what else to add to it.
Closes #129