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

[MIRROR] Adds missing gas senders to the North Star #2129

Merged
merged 1 commit into from
Feb 26, 2024

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Nova: NovaSector/NovaSector#1135
Original PR: tgstation/tgstation#81652

About The Pull Request

Adds a sender to the atmos project room, and bluespace gas vendors to each floor in the main halls + a bonus one in the SM room.

Why It's Good For The Game

Fixes: #80988

Changelog

🆑 Cheshify
fix: The North Star finally has a bluespace gas sender system.
/:cl:

* Adds missing gas senders to the North Star (#81652)

## About The Pull Request
Adds a sender to the atmos project room, and bluespace gas vendors to
each floor in the main halls + a bonus one in the SM room.
## Why It's Good For The Game
Fixes: #80988
## Changelog
:cl:
fix: The North Star finally has a bluespace gas sender system.
/:cl:

* Adds missing gas senders to the North Star

---------

Co-authored-by: Cheshify <[email protected]>
@Iajret Iajret merged commit 1c4cb04 into master Feb 26, 2024
24 checks passed
@Iajret Iajret deleted the upstream-mirror-1135 branch February 26, 2024 19:38
AnywayFarus added a commit that referenced this pull request Feb 26, 2024
Iajret pushed a commit that referenced this pull request Apr 22, 2024
* Fixes Alien Numerical Identifier Transfer System (#82762)

## About The Pull Request

Fixes #82756

I'm not sure what the root cause of the issue is or why it broke so
recently (this has been a thing since #49150) but I went and cleaned up
the code and ensured that we always reset the `name` to the `initial()`
value we have for it if we determine that we need to transfer over the
`numba` from the old xenomorph. This is done by a new proc which is a
lot less convoluted from the old system (always respecting var-edited
names and stuff like that) that should make it flow a lot better and
make it more hardy against double-addition of the `numba` to the name.
## Why It's Good For The Game

Having two numbers in your name is weird, should only have one.

[x] I tested this PR
## Changelog
:cl:
fix: Evolved aliens should no longer have two numbers in their name.
/:cl:

* Fixes Alien Numerical Identifier Transfer System

---------

Co-authored-by: san7890 <[email protected]>
ReezeBL pushed a commit that referenced this pull request Apr 22, 2024
)

* Fixes Alien Numerical Identifier Transfer System (#82762)

## About The Pull Request

Fixes #82756

I'm not sure what the root cause of the issue is or why it broke so
recently (this has been a thing since #49150) but I went and cleaned up
the code and ensured that we always reset the `name` to the `initial()`
value we have for it if we determine that we need to transfer over the
`numba` from the old xenomorph. This is done by a new proc which is a
lot less convoluted from the old system (always respecting var-edited
names and stuff like that) that should make it flow a lot better and
make it more hardy against double-addition of the `numba` to the name.
## Why It's Good For The Game

Having two numbers in your name is weird, should only have one.

[x] I tested this PR
## Changelog
:cl:
fix: Evolved aliens should no longer have two numbers in their name.
/:cl:

* Fixes Alien Numerical Identifier Transfer System

---------

Co-authored-by: NovaBot <[email protected]>
Co-authored-by: san7890 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants