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

word bomb mini rework #163

Open
wants to merge 48 commits into
base: master
Choose a base branch
from
Open

word bomb mini rework #163

wants to merge 48 commits into from

Conversation

Jeftaei
Copy link
Contributor

@Jeftaei Jeftaei commented Jul 18, 2024

Hopefully its finally done, might be missing a few commits from master i forgor

  • internally reworks word bomb mini to try and make it more generic and have support for future games and making word bomb mini easier to develop
  • reworks remarks to make them easier to edit and add upon
  • slightly changes leaderboard functionality for future use
  • changes dictionaries to support sets (in most cases) to hopefully make that section a bit faster
  • other various qol stuff
    • sendReplyWithID
    • isPlural
    • save state for word bomb mini
    • partially types db.ts

@iminlikewithyou iminlikewithyou linked an issue Jul 19, 2024 that may be closed by this pull request
@iminlikewithyou
Copy link
Member

i think generatePrompt (in dictionary.ts) is where the option to change the minimum amount of solutions should be rather than enforcing it when generating the prompt frequency map

@iminlikewithyou
Copy link
Member

gotta make sure the new generator can make something similar to this, or at least make prompts that people enjoy Lol

https://docs.google.com/spreadsheets/d/1tzU88qNVsUWna8mbLno3Ywb60OOLE8CQ-0vNSlBV0GA/edit?gid=0#gid=0

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.

Stop supporting ? as a wildcard search character
2 participants