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] Deletes unused hydroponics code #1015

Merged
merged 1 commit into from
Dec 7, 2023
Merged

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Skyrat: Skyrat-SS13/Skyrat-tg#25480
Original PR: tgstation/tgstation#80156

About The Pull Request

/obj/item/seeds/sample was completely unused, ever since it's inception in #16082 (tgstation/tgstation@d513f52) as far as I can tell

(Me when "adds plant samples for future use" never makes use of them in the future)

All this does is complicate hydro code so it dies.
Anything it is doing should be handled by genes probably

While I was here I made sure nothing in can_add will break and touched up the code a tiny tiny bit. I kept it as SHOULD_CALL_PARENT even though it doesn't make use of it at the moment because every trait currently calls parent / respects return value, so sure, why not

Changelog

Not necessary

* Deletes unused hydroponics code (#80156)

## About The Pull Request

`/obj/item/seeds/sample` was completely unused, ever since it's
inception in #16082
(tgstation/tgstation@d513f52)
as far as I can tell

(Me when "adds plant samples for future use" never makes use of them in
the future)

All this does is complicate hydro code so it dies.
Anything it is doing should be handled by genes probably

While I was here I made sure nothing in `can_add` will break and touched
up the code a tiny tiny bit. I kept it as `SHOULD_CALL_PARENT` even
though it doesn't make use of it at the moment because every trait
currently calls parent / respects return value, so sure, why not

## Changelog

Not necessary

* Deletes unused hydroponics code

---------

Co-authored-by: MrMelbert <[email protected]>
@Iajret Iajret merged commit 4f576bf into master Dec 7, 2023
23 checks passed
@Iajret Iajret deleted the upstream-mirror-25480 branch December 7, 2023 11:27
Iajret pushed a commit that referenced this pull request Feb 18, 2024
…ton (#1015)

* Changes the crystallizer wrench screentip to the correct button (#81516)

## About The Pull Request
it says left click, that does nothing, you need to right click it to
rotate
## Why It's Good For The Game
correct ingame information
## Changelog
:cl:
fix: The crystallizer screentip for rotating it has been updated with
the correct button
/:cl:

* Changes the crystallizer wrench screentip to the correct button

---------

Co-authored-by: 1393F <[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