Skip to content

feat: bot rewrite#1

Open
OfficialSirH wants to merge 14 commits intomainfrom
clean-rewrite
Open

feat: bot rewrite#1
OfficialSirH wants to merge 14 commits intomainfrom
clean-rewrite

Conversation

@OfficialSirH
Copy link

I've got a few things to still change before I consider this PR finished

The following were done:

  • fixed things that weren't done so well or literally didn't work at all
  • Made stuff a little more easier to read through
  • moved a lot of the kings decree functionality into a static class for a more clear way of interacting with the game's key components

@OfficialSirH
Copy link
Author

I'll fix those actions later as well, they're currently just a copy-paste from AutoModerator

Copy link
Member

@didinele didinele left a comment

Choose a reason for hiding this comment

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

initial nits

@OfficialSirH
Copy link
Author

hmmm, I wonder if we should either merge this right now and worry about its current issues next year or fix all of the issues it has and then merge it 🤔

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.

3 participants