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

Remove cheat support? #111

Closed
freiheit opened this issue Mar 25, 2024 · 0 comments · Fixed by #112
Closed

Remove cheat support? #111

freiheit opened this issue Mar 25, 2024 · 0 comments · Fixed by #112
Assignees
Labels
good first issue Good for newcomers

Comments

@freiheit
Copy link
Owner

Not useful and complicates code.

Basically, was sorta fun to have that option at first, but I'm ready to drop it.

@freiheit freiheit added the good first issue Good for newcomers label Mar 25, 2024
@freiheit freiheit self-assigned this Mar 25, 2024
freiheit added a commit that referenced this issue Mar 25, 2024
@freiheit freiheit linked a pull request Mar 25, 2024 that will close this issue
freiheit added a commit that referenced this issue Mar 25, 2024
freiheit added a commit that referenced this issue Mar 25, 2024
* Strip out cheating code.
Closes #111

* Pull current test stuff into main branch (#114)

* 106 Add a unit test class; add tests for the parser
* Add a test that "d10 d10" == "2d10"
* Helper script for test.py
* Add a github action to run tests

Co-authored-by: @slashingweapon
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant