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

Counts more stuff as suicide #4876

Conversation

Arthur-Holiday
Copy link
Contributor

[BALANCE]

About the PR

Makes it so more methods of killing yourself count as suicide, I didn't touch some stuff on porpouse as those require more effort than going to the bar and grabbing a russian revolver or going to the chapel and grabbing a bible, if it should all count just tell me.

Why's this needed?

Being able to easily blow up somewhere with no risk of it not working by using micro/macro bombs and something easily obtainable is bad.

@keywordlabeler keywordlabeler bot added the C-Balance Balance changes, buffs and nerfs label May 25, 2021
@github-actions github-actions bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label May 25, 2021
@Studenterhue Studenterhue added the E-Add-To-Wiki A PR that will require changes to the wiki label May 25, 2021
@Camryn-Buttes
Copy link
Contributor

Camryn-Buttes commented May 25, 2021

Good idea, but I do have an issue with the bible fart being classified as a suicide since there are multiple ways to force someone to fart on a bible or to trick them into farting on it (there's even a miniature bible whose smaller sprite is specifically intended to facilitate this trickery)

EDIT: on second thought, no, for the same reasons penny elaborated

@penny64
Copy link
Contributor

penny64 commented May 25, 2021

I don't think this makes any sense, the only time there's metaphysical effects on your death is when you use the 'suicide' verb.
It's only expected that the rules change when you activate this function.
How is a player to know that a Bible is or isn't "suicide"?
Or a Russian revolver?
When a new user implants themselves with a micromb and grabs a Russian revolver, how will they understand why they died on accident but didn't explode comedically?
How does a player know a bomb won't just count as suicide?
Mechanically all of these should work until you press the button that changes the rules, this breaks the contract

user.TakeDamage("head", 300, 0)
take_bleeding_damage(user, null, 500, DAMAGE_STAB)
SPAWN_DBG(50 SECONDS)
Copy link
Contributor

Choose a reason for hiding this comment

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

Why 50 seconds?

@PrimeNumb
Copy link
Contributor

I don't think this makes any sense, the only time there's metaphysical effects on your death is when you use the 'suicide' verb.
It's only expected that the rules change when you activate this function.
How is a player to know that a Bible is or isn't "suicide"?
Or a Russian revolver?
When a new user implants themselves with a micromb and grabs a Russian revolver, how will they understand why they died on accident but didn't explode comedically?
How does a player know a bomb won't just count as suicide?
Mechanically all of these should work until you press the button that changes the rules, this breaks the contract

Agreed, this change would make it very unintuitive and ambiguous.

Also, please be a little more descriptive in the PR, i.e. list every suicide that you touched. This makes it so people can quickly get a better grasp of what your PR actually changes, without having to poke around in the code to figure out what you did. Poking around in the code is better suited for reviewing how you implemented your changes rather than what you changed.

@Arthur-Holiday
Copy link
Contributor Author

Penny is right, until I think of some better way to do this, wich I don't think there is, I would rather It not be a thing at all, would generate a lot of confusion.

@BatElite BatElite removed the E-Add-To-Wiki A PR that will require changes to the wiki label May 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Balance Balance changes, buffs and nerfs size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants