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

Upstream 1(fixed) #80

Merged
merged 183 commits into from
Nov 24, 2024
Merged

Upstream 1(fixed) #80

merged 183 commits into from
Nov 24, 2024

Conversation

AwareFoxy
Copy link
Collaborator

@AwareFoxy AwareFoxy commented Nov 24, 2024

апстрим

Jezithyr and others added 30 commits October 12, 2024 22:21
It's simple. We kill the heisentest
It's simple. We kill the heisentest
* Fix some rounds failing to end due to mind roles

Fixes #32791

This is caused by ShowRoundEndScoreboard running into a bug trying to display antags: some player is showing up as antag with MindIsAntagonist(), but has no antag roles listed in MindGetAllRoleInfo().

This was caused by one of the roles of the player having the Antag boolean set, but having no AntagPrototype set.

The responsible mind role appeared to be MindRoleSubvertedSilicon which is missing a set for the SubvertedSilicon antag prototype.

I also added resilience to the round-end code to make it so that an exception showing the scoreboard (and sending the Discord message) would not cause the round end logic to completely abort from an exception.

I am planning to add an integration test to cover this bug (no prototype in mind roles), but I'll leave that for not-the-immediate-hotfix.

* At least one maintainer approved this tiny PR without reading it, not naming names.
Fix tech anomaly nexttimer (#32805)

Co-authored-by: metalgearsloth <[email protected]>
* fix ninja bomb component check

* remove TryGetRole
Fix: Plushies no longer delete items when recycled (#32838)

fix

Co-authored-by: beck-thompson <[email protected]>
Update submodule

This fixes an important memory leak.
…tem. (#32942)

Fix loneop spawnrate by reverting it to not use the custom shuttle event system.
* Add debug messages to traitor activation

* more debug
* Initial commit

* Review feedback changes

* ProtoId

* TempCommit

* First attempt to have client alerts

* Review changes
* jumpsuit detailening

* jumpskirt stuff

* meta.json

* update meta.json

* meta.json fix fix

* meta.json fix fix fix
* init

* cleanup

* Oops! Forgot something

* addressing changes

* guh

* guh 2.0

* some cleanup

* all bless the intellicard

* Yippee

* small locale thing

* changes + small bugfix

---------

Co-authored-by: slarticodefast <[email protected]>
@AwareFoxy AwareFoxy requested a review from Morb0 as a code owner November 24, 2024 19:59
@github-actions github-actions bot added S: Needs Review ПРу необходим ревью S: Untriaged size/L Changes: Sprites Changes: Map Изменяет карты Changes: Localization Изменяет локализацию Changes: UI and removed size/L labels Nov 24, 2024
@AwareFoxy AwareFoxy changed the title Upstream Upstream 1(fixed) Nov 24, 2024
@Vonsant Vonsant merged commit 5a5650e into space-syndicate:master Nov 24, 2024
26 checks passed
@AwareFoxy AwareFoxy deleted the upstream branch December 15, 2024 00:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Localization Изменяет локализацию Changes: Map Изменяет карты Changes: Sprites Changes: UI S: Needs Review ПРу необходим ревью S: Untriaged
Projects
None yet
Development

Successfully merging this pull request may close these issues.