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

feat: bot rewrite #1

Open
wants to merge 14 commits into
base: main
Choose a base branch
from
Open

feat: bot rewrite #1

wants to merge 14 commits into from

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

.github/workflows/deploy.yml Outdated Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
.github/workflows/test.yml Outdated Show resolved Hide resolved
Dockerfile Outdated Show resolved Hide resolved
Dockerfile Outdated Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
@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