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

Prevent welders cauterising on harm #5882

Conversation

flappybatpal
Copy link
Contributor

@flappybatpal flappybatpal commented Sep 2, 2021

[QOL]

About the PR

Lit welders on harm intent always do damage.

Why's this needed?

Healing action on a Harm intent makes little sense to a user.
Simplifies using welders for damage rather than a unique interaction that can be difficult to understand in combat. Welders awkwardly alternate between damage and cauterizing if the target was bleeding.
Limb targeting with a lit welder would always try to cauterize if the target was missing a limb, using an unlit welder or anything else would do damage.

Changelog

(u)Flappybat
(+)Welders will always do damage on harm intent rather than trying to cauterize bleeding.

@keywordlabeler keywordlabeler bot added the C-QoL A quality of life improvement that makes the game easier to play label Sep 2, 2021
@github-actions github-actions bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Sep 2, 2021
Copy link
Contributor

@flrsh flrsh left a comment

Choose a reason for hiding this comment

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

Makes sense to me.

@UrsulaMejor
Copy link
Contributor

yeah

@UrsulaMejor UrsulaMejor merged commit 544b750 into goonstation:master Sep 2, 2021
github-actions bot pushed a commit that referenced this pull request Sep 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-QoL A quality of life improvement that makes the game easier to play size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants