Skip to content

Fix frenzy stab #151

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

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

Fix frenzy stab #151

wants to merge 2 commits into from

Conversation

derNarr
Copy link
Contributor

@derNarr derNarr commented Jun 10, 2021

This is a first attempt to allow for a stab after a block on a frenzy player.

Tests go through smoothly now, but I have to do some manual testing with this as well.

Code for Blitz is still missing.

(This PR also contains the fix for the throw teammate as I branched from the wrong starting branch -- I might fix this, if the throw teammate PR is not merged soon.)

@derNarr
Copy link
Contributor Author

derNarr commented Jun 12, 2021

This one is not correct yet. It allows to stab anyone after the first Frenzy DO NOT MERGE :-)

@njustesen
Copy link
Owner

What's the status on this one? Seems like good work but not quite there?

@njustesen njustesen added the Stale label Sep 3, 2021
@derNarr
Copy link
Contributor Author

derNarr commented Sep 3, 2021

I did not cracked it, but now after some time is gone by, I might give it a fresh look. I let you know, if I make any progress. As this was a TODO in the code, I thought I share my still unsuccessful steps in this direction.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants