Skip to content
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

Refactored Help command #22

Merged
merged 10 commits into from
Jul 28, 2021
Merged

Refactored Help command #22

merged 10 commits into from
Jul 28, 2021

Conversation

Lolispo
Copy link
Owner

@Lolispo Lolispo commented Jul 28, 2021

(Based on other pull request - merge first)
Help commands moved to new format
Had to split them up due to Discord limitation, refactored code so that was easier
Added so help detailed functionality is implementable (so we can write more detailed help explanations if we want to. These are seen by the helpall command).
Tweaks from the previous refactor of commands
Moved types to a new file for the BaseCommands to follow that convention
Added ability to set if a message should be parsable in direct message or not (Supply the types you want) - So help command is example of this (works in server and DM)

@Lolispo
Copy link
Owner Author

Lolispo commented Jul 28, 2021

Work in progress - found that some tests failed and then wanted to activate strict mode. (test was circular dependency + added hej test)

src/BaseCommand.ts Show resolved Hide resolved
src/BaseCommand.ts Show resolved Hide resolved
src/BaseCommand.ts Show resolved Hide resolved
src/bot.ts Show resolved Hide resolved
src/bot.ts Show resolved Hide resolved
src/tools/f.ts Show resolved Hide resolved
src/tools/f.ts Show resolved Hide resolved
Copy link
Collaborator

@Samev Samev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Haven't checked "WIP:strict, hej test - get all tests to pass and build to run" and onward.

src/game/gameModes.ts Outdated Show resolved Hide resolved
src/game/gameModes.ts Show resolved Hide resolved
@Lolispo Lolispo merged commit 2e063ce into master Jul 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants