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: Fix some invalid damage types #288

Conversation

SimpleStation14
Copy link
Member

Mirror of PR #26295: Fix some invalid damage types from space-wizards space-wizards/space-station-14

b34777177ca2b379c67fb5d0d117f95b8809f43e

PR opened by Tayrtahn at 2024-03-20 19:10:30 UTC


PR changed 7 files with 8 additions and 8 deletions.

The PR had the following labels:

  • No C#

Original Body

About the PR

Corrected the names of a few damage types in various DamageTypeSpecifiers.

Why / Balance

This might have some balance implications, as a few things will be doing damage they didn't previously.

  • Space bears now do some blunt damage along with the usual slash.
  • The Throngler does massive structural damage.
  • RobustHarvest does a little heat damage to non-plants.
  • Fresium does a bit of negative heat damage.

All of these effects were presumably intended by the authors of various PRs that added them, but were nonfunctional due to using an invalid name. If it's decided that these additional damage types are undesired, they should just be removed instead of relying on a bug to ignore them.

I also changed a few entries that were doing 0 damage in invalid types. They now do 0 damage in valid types, for whatever that's worth.

Technical details

I haven't figured out how to get the YAML linter to properly catch these issues, but I got it to spit out a list of these errors while messing around with it, so I figured I'd go ahead and fix them.

Media

  • I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

@SimpleStation14 SimpleStation14 added the Pull Request Mirror Mirrors a PR from another Repo. Automatically applied by mirror bot label Apr 22, 2024
@SimpleStation14 SimpleStation14 marked this pull request as draft May 4, 2024 21:12
@VMSolidus VMSolidus marked this pull request as ready for review May 12, 2024 03:16
@VMSolidus VMSolidus merged commit a792380 into Simple-Station:master May 12, 2024
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Pull Request Mirror Mirrors a PR from another Repo. Automatically applied by mirror bot
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants