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

Revert "Revert "Fix UninitializedSaveTest prototypes" (#16169)" #16175

Merged
merged 2 commits into from
May 6, 2023

Conversation

ElectroJr
Copy link
Member

I haven't actually managed to figure out how to reproduce the crash, but it might be fixed now?
It would be good if If someone could figure out how to actually reproduce it.

@keronshb
Copy link
Contributor

keronshb commented May 6, 2023

Hold off on merging, reviewing this

@keronshb keronshb added the S: DO NOT MERGE Status: Open item that should NOT be merged. DNM. Allows test to run unlike draft. label May 6, 2023
@keronshb
Copy link
Contributor

keronshb commented May 6, 2023

Content.Client_0u6MhfbjHt.mp4

Tested, spammed lightning, nothing catastrophic.

@keronshb keronshb added S: Needs Review Status: Requires additional reviews before being fully accepted and removed S: DO NOT MERGE Status: Open item that should NOT be merged. DNM. Allows test to run unlike draft. labels May 6, 2023
@keronshb
Copy link
Contributor

keronshb commented May 6, 2023

This should be fine since lightning isn't going ham anymore.

@keronshb keronshb merged commit e379322 into space-wizards:master May 6, 2023
rbertoche pushed a commit to rbertoche/space-station-14 that referenced this pull request May 25, 2023
…16169)" (space-wizards#16175)

* Revert "Revert "Fix UninitializedSaveTest prototypes" (space-wizards#16169)"

This reverts commit de1613f.

* Invert if statement
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S: Needs Review Status: Requires additional reviews before being fully accepted
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants