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

Fix heli Sabotage #45

Merged
merged 2 commits into from
Dec 26, 2023
Merged

Fix heli Sabotage #45

merged 2 commits into from
Dec 26, 2023

Conversation

NikoCat233
Copy link
Contributor

@NikoCat233 NikoCat233 commented Dec 18, 2023

It never actually works
Also check whether mushroom mixup is active before sabotage

@scp222thj
Copy link
Owner

scp222thj commented Dec 26, 2023

@NikoCat233 Thanks for fixing the Heli sabotage, I had no idea it was having issues! However, the fact that MushroomMixup sabotage can be re-activated while it is already running is intended, because it allows users to reset the timer and start all over again. I will revert the MushroomMixup changes, and then I think I'll be able to merge your pull request with no issues!

@scp222thj scp222thj merged commit 392c2b5 into scp222thj:main Dec 26, 2023
1 check passed
@NikoCat233
Copy link
Contributor Author

Guess ok

@NikoCat233 NikoCat233 deleted the KeepWithMain branch December 27, 2023 12:44
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.

2 participants