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] Fixes a spurious runtime off loading the museum away mission #2035

Merged
merged 1 commit into from
Feb 19, 2024

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Nova: NovaSector/NovaSector#1020
Original PR: tgstation/tgstation#81520

About The Pull Request

It loaded a random spawner on top of a chasm. The spawner has a chance to drop the dust decal, which errors when trying to spawn on a chasm

Yayeeeeeet

…#1020)

* Fixes a spurious runtime off loading the museum away mission (#81520)

## About The Pull Request

It loaded a random spawner on top of a chasm. The spawner has a chance
to drop the dust decal, which errors when trying to spawn on a chasm

Yayeeeeeet

* Fixes a spurious runtime off loading the museum away mission

---------

Co-authored-by: LemonInTheDark <[email protected]>
@Iajret Iajret merged commit b8508a7 into master Feb 19, 2024
24 checks passed
@Iajret Iajret deleted the upstream-mirror-1020 branch February 19, 2024 13:27
Iajret pushed a commit that referenced this pull request Apr 18, 2024
…as obj/item, instead of being just a var (#2035)

* Quirks, which give items, now have quirk_item arg specified as obj/item, instead of being just a var (#82650)

## About The Pull Request
quirk_item is now /obj/item, since it will allow for calling procs or
getting variables from this item

It's required for non-modular translation to call for item's name to
remove articles

## Why It's Good For The Game
It's always an item, and if it's a path, it's already checked for it.
Better usage in the future.

* Quirks, which give items, now have quirk_item arg specified as obj/item, instead of being just a var

---------

Co-authored-by: larentoun <[email protected]>
ReezeBL pushed a commit that referenced this pull request Apr 18, 2024
…as obj/item, instead of being just a var (#2035) (#2937)

* Quirks, which give items, now have quirk_item arg specified as obj/item, instead of being just a var (#82650)

## About The Pull Request
quirk_item is now /obj/item, since it will allow for calling procs or
getting variables from this item

It's required for non-modular translation to call for item's name to
remove articles

## Why It's Good For The Game
It's always an item, and if it's a path, it's already checked for it.
Better usage in the future.

* Quirks, which give items, now have quirk_item arg specified as obj/item, instead of being just a var

---------

Co-authored-by: NovaBot <[email protected]>
Co-authored-by: larentoun <[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