Skip to content
This repository was archived by the owner on May 22, 2025. It is now read-only.

Removes sloth ruin#10225

Closed
Takahiru wants to merge 1 commit into
yogstation13:masterfrom
Takahiru:sloth
Closed

Removes sloth ruin#10225
Takahiru wants to merge 1 commit into
yogstation13:masterfrom
Takahiru:sloth

Conversation

@Takahiru

Copy link
Copy Markdown
Contributor

Intent of your Pull Request

The sloth ruin is simply a waste of space on lavaland. It provides nothing except maybe tricking a newbie miner one time only for them to never enter it again.
I tried thinking of improving it, but sloth is literally the most boring of the seven sins so I can't think of anything.

Changelog

🆑
rscdel: Removes sloth ruin
/:cl:

@Takahiru Takahiru requested a review from a team October 28, 2020 21:24
@Takahiru Takahiru requested a review from a team as a code owner October 28, 2020 21:24
@Yogbot-13 Yogbot-13 added DME Edit This PR affects the yogstation.DME file Mapping Will result in a change to a map. Revert / Deletion Altoids is probably mad about this labels Oct 28, 2020
@ynot01

ynot01 commented Oct 28, 2020

Copy link
Copy Markdown
Contributor

sloth is literally the most boring of the seven sins so I can't think of anything.

this almost makes me want to keep it for the irony

@github-actions github-actions Bot added the Config Config files need to be changed on the host for this to work label Oct 28, 2020
@boodaliboo

Copy link
Copy Markdown
Contributor

keep it

@fluffe9911

Copy link
Copy Markdown
Contributor

eh Ill get around to merging this eventually maybe meh

@patpol4 patpol4 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't remove it from the game, blacklist it.

@MrHorizons

Copy link
Copy Markdown
Contributor

I think we should keep it, if only to keep the "sins" theme whole. Though we could certainly cut the size of the ruin down heavily: perhaps they didn't even have a ruin to add,

@Takahiru

Copy link
Copy Markdown
Contributor Author

I think we should keep it, if only to keep the "sins" theme whole. Though we could certainly cut the size of the ruin down heavily: perhaps they didn't even have a ruin to add,

It seems like they did intend to expand on it based on one of the comments, but it never happened.

@Takahiru

Copy link
Copy Markdown
Contributor Author

Don't remove it from the game, blacklist it.

its in the blacklist config by default e

@UselessTheremin

Copy link
Copy Markdown
Contributor

the point is that it sucks and is a waste of space. i find it funny, and it helps balance out how there are really good ruins.

@MegaEmpirical

Copy link
Copy Markdown

Not all ruins need to be 'good'. Miners have enough loot.

@Takahiru

Copy link
Copy Markdown
Contributor Author

Not all ruins need to be 'good'. Miners have enough loot.

It dosen't even look good, it's just a waste of space, i didn't want to give them another item either so idk

@MrHorizons MrHorizons mentioned this pull request Oct 29, 2020
@Takahiru

Copy link
Copy Markdown
Contributor Author

brr people really think its a great part of the game so goodbye pr

@Takahiru Takahiru closed this Oct 29, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Config Config files need to be changed on the host for this to work DME Edit This PR affects the yogstation.DME file Mapping Will result in a change to a map. Revert / Deletion Altoids is probably mad about this

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants