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 cigarettes and tendrils #1490

Merged
merged 2 commits into from
Mar 26, 2024
Merged

Fix cigarettes and tendrils #1490

merged 2 commits into from
Mar 26, 2024

Conversation

Absolucy
Copy link
Member

@Absolucy Absolucy commented Mar 26, 2024

About The Pull Request

Why It's Good For The Game

Changelog

🆑
fix: Fixed cigarettes not being smokeable without a space helmet + internals.
fix: Fixed being unable to melee lavaland tendrils.
/:cl:

Jacquerel and others added 2 commits March 26, 2024 19:19
## About The Pull Request

Fixes #78199 
PR #78122 (ef44be8) was intended to allow you to smoke cigarettes in space if you
have an oxygenated helmet on.
What it _actually_ did was make it so you can _only_ smoke cigarettes if
you have an oxygenated helmet on, regardless of whether or not you are
in space (or if you inject them with liquid oxygen).

I moved the shared check into a proc to reduce duplicate code and fixed
the logic.

## Changelog

:cl:
fix: It is now possible to smoke cigarettes even if you aren't wearing a
safety helmet
/:cl:
## About The Pull Request
adds an `else . = ..()` to spawner's attackby check so you can just
break them with your crusher in case you need to be rid of it

## Why It's Good For The Game

sometimes you just need to melee a spawner

## Changelog

:cl:
fix: Necropolis tendrils and other mining mob spawners can be hit in
melee again.
/:cl:

---------

Co-authored-by: Hatterhat <[email protected]>
@dwasint dwasint merged commit f3dee5d into Monkestation:master Mar 26, 2024
8 checks passed
github-actions bot added a commit that referenced this pull request Mar 26, 2024
@Absolucy Absolucy deleted the fixy-wixy branch March 26, 2024 23:21
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.

4 participants