-
Notifications
You must be signed in to change notification settings - Fork 12
Strike system improvements #462
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
Merged
Merged
Changes from 32 commits
Commits
Show all changes
40 commits
Select commit
Hold shift + click to select a range
e4b1cfe
implement get strikes command
MattyTheHacker 62b1e84
move the ignore
MattyTheHacker 0503c78
Fix this
MattyTheHacker d094358
Fix using wrong field
MattyTheHacker 826a25b
try something else
MattyTheHacker a0d3d22
Add some debug
MattyTheHacker 92100df
fix mypy
MattyTheHacker e5b5ffe
getting better
MattyTheHacker bfab1d2
i literally hate these precommits
MattyTheHacker c265d58
minor refactor
MattyTheHacker 6795cef
Add delete-all strikes command
MattyTheHacker 0ab4fa4
refactor to handle exceptions
MattyTheHacker 08ec83f
make response ephemeral
MattyTheHacker caa2a66
Implement decrement strike command
MattyTheHacker b0ac12d
fix mypy
MattyTheHacker 0de9e42
Add response for decremenet
MattyTheHacker 4fefdb0
Update strike.py
MattyTheHacker cb13344
Merge branch 'main' into 71-strikes-functionality
MattyTheHacker cd6c8d5
Merge branch 'main' into 71-strikes-functionality
MattyTheHacker 6f76d54
Merge branch 'main' into 71-strikes-functionality
MattyTheHacker fee87e2
Merge branch 'main' into 71-strikes-functionality
MattyTheHacker 17805bd
Merge branch 'main' into 71-strikes-functionality
MattyTheHacker 512f70d
Merge branch 'main' into 71-strikes-functionality
MattyTheHacker e9c778c
Merge branch 'main' into 71-strikes-functionality
MattyTheHacker d6c4976
Merge branch 'main' into 71-strikes-functionality
MattyTheHacker 8b54c46
Merge branch 'main' into 71-strikes-functionality
MattyTheHacker 3ed4a50
Merge branch 'main' into 71-strikes-functionality
MattyTheHacker a7fae4d
improve fetching of strike object
MattyTheHacker 62786d7
add logging
MattyTheHacker ef7d3ab
minor fixes
MattyTheHacker fd5f42f
fix decrement object fetch
MattyTheHacker a945af9
[pre-commit.ci lite] apply automatic fixes
pre-commit-ci-lite[bot] 186fee6
update old method too
MattyTheHacker ab7fac7
Merge branch 'main' into 71-strikes-functionality
MattyTheHacker e6b7f11
fix ephemerality
MattyTheHacker b60617f
[pre-commit.ci lite] apply automatic fixes
pre-commit-ci-lite[bot] 1837306
fix the f string stuff
MattyTheHacker ac25207
yeet the import
MattyTheHacker c2bbfbb
Merge branch 'main' into 71-strikes-functionality
MattyTheHacker 3293478
revert to use `asave`
MattyTheHacker File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.