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 hugger/lesser drone time of death bypass #4334

Merged
merged 1 commit into from
Sep 5, 2023

Conversation

RenaRenaRe
Copy link
Contributor

@RenaRenaRe RenaRenaRe commented Sep 4, 2023

About the pull request

Should fix the bypass_time_of_death_checks variable which is supposed to allow facehuggers and lesser drones to be eligible for larva queue even if they have recently died.
NOTE: I have no idea how to test this locally since for some reason I just cannot get larva queue to work locally at all (even when I haven't changed any code). I figured I'd put this PR up anyway and maintainers can take a look at it, but it may or may not actually work.

Explain why it's good for the game

It was clearly intended that ghosts of lesser drones and face huggers (and anything else with bypass_time_of_death_checks set to true) will bypass the time of death check when the game checks for larva queue candidates. I think because of how the logic was written it never actually worked properly unless you were an admin.

Testing Photographs and Procedure

Screenshots & Videos

Put screenshots and videos here with an empty line between the screenshots and the <details> tags.

Changelog

🆑
fix: Hugger and Lesser Drone ghosts now actually bypass time of death checks for the larva queue.
/:cl:

@github-actions github-actions bot added the Fix Fix one bug, make ten more label Sep 4, 2023
@morrowwolf morrowwolf added this pull request to the merge queue Sep 5, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Sep 5, 2023
@morrowwolf morrowwolf added this pull request to the merge queue Sep 5, 2023
Merged via the queue into cmss13-devs:master with commit 4f28269 Sep 5, 2023
27 checks passed
cm13-github added a commit that referenced this pull request Sep 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix Fix one bug, make ten more
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants