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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

[MIRROR] [PORT] Ports tgstation/tgstation#51453 real quick #2739

Merged
merged 2 commits into from
Dec 17, 2020

Conversation

AustationBot
Copy link
Member

Original PR: BeeStation/BeeStation-Hornet#3149

About The Pull Request

Ports tgstation/tgstation#51453

I can't tell with 100% certainty (I am 75% certain that it caused currently unreported problems) if #3089 fucked Beecode, I didn't get reports about it, but I want to make sure, and I didn't like internal pumps from the moment I saw it in the first place; it created ridiculous amount of useless pipenets: about 300.

This PR also includes suppression of warning-like runtimes happening while init dir caching, which I didn't even know til I investigated a runtime reported by Archanial, but this part is trivial.

There are other pipenet fixes (hint: that won't be porting tgstation/tgstation#55514) I have in mind (read: still can't decide how to fix), but I am sure they can wait, and I am going to a start week-long hiatus soon(tm). I am not going to include them here.

Why It's Good For The Game

This should result in less broken atmos. But... is "less broken atmos" possible???

Changelog

馃啈 nemvar
fix: fixed atmos component code (again)
/:cl:

park66665 and others added 2 commits December 16, 2020 01:20
* Ports tgstation/tgstation#51453 real quick

* Fixes turf atmos getting deleted.

* Moves the gas transfer procs to the gas mixture.

* Fixes emp act to behave like before

* Fixes tanks exploding

* prevents pipenet related warnings

Co-authored-by: nemvar <47324920+nemvar@users.noreply.github.com>
@AustationBot AustationBot added Fix Fixes a bug Port Something copied from another repository Upstream PR Merged This PR is mirrored from upstream. Automatically assigned by the AuStation Bot. labels Dec 15, 2020
@TheFakeElon TheFakeElon merged commit bc05ca7 into master Dec 17, 2020
@TheFakeElon TheFakeElon deleted the upstream-merge-3149 branch December 17, 2020 15:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix Fixes a bug Port Something copied from another repository Upstream PR Merged This PR is mirrored from upstream. Automatically assigned by the AuStation Bot.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants