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

Update SetOutfitCommand.cs #941

Merged

Conversation

VMSolidus
Copy link
Member

Description

I hate when people don't use nullables correctly, and SOMEHOW a null reference is still being handed into IPCs.

@github-actions github-actions bot added the Changes: C# Changes any cs files label Sep 20, 2024
@VMSolidus
Copy link
Member Author

I'm going to rerun the tests on this like 5 times in a row to see if this actually fixes the stupid Heisentest.

@FoxxoTrystan FoxxoTrystan added Priority: 1-Urgent Must be resolved immediately Size: 5-Very Small For especially small issues/PRs labels Sep 20, 2024
@FoxxoTrystan FoxxoTrystan requested review from a team, FoxxoTrystan, DEATHB4DEFEAT, Peptide90, Pspritechologist and OldDanceJacket and removed request for a team September 20, 2024 19:35
@FoxxoTrystan
Copy link
Member

Honk, no more games!

@github-actions github-actions bot added the Status: Needs Review Someone please review this label Sep 20, 2024
@VMSolidus VMSolidus merged commit 981c4f6 into Simple-Station:master Sep 20, 2024
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: C# Changes any cs files Priority: 1-Urgent Must be resolved immediately Size: 5-Very Small For especially small issues/PRs Status: Needs Review Someone please review this
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants