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

Allow a heavy training dummy in place of light #77905

Merged

Conversation

AlexMooney
Copy link
Contributor

@AlexMooney AlexMooney commented Nov 16, 2024

Summary

Content "Allow using a heavy training dummy where a light one will suffice"

Purpose of change

The standard training dummy is made of wood, nails, and duct tape, along with some markings. The armored training dummy is the same thing with some scrap metal armor on top. I think that if you're practicing how to aim your strikes with a weapon, the armored dummy should suffice. I don't think that the training consists of punching or shivving a hunk of wood with force because that would damage your limbs or break your shiv, so since the training doesn't involve hard contact, the armored dummy should work.

Describe the solution

Add pseudo_training_dummy_heavy to the couple of places that have pseudo_training_dummy_light without it.

Describe alternatives you've considered

Leaving it alone. Changing the shiv practice to consume many shivs and possibly try to make the unarmed practice inflict damage.

Testing

I don't have a functional dev environment for CDDA, but as you can see the JSON change is trivial.

Additional context

Upgrading your training dummy without finishing training shivs or unarmed creates a chore without any particular purpose.

Considering the dummy is made of wood, it's not going to be especially
soft to strike a "light" training dummy.
@github-actions github-actions bot added [JSON] Changes (can be) made in JSON Melee Melee weapons, tactics, techniques, reach attack astyled astyled PR, label is assigned by github actions json-styled JSON lint passed, label assigned by github actions labels Nov 16, 2024
@Night-Pryanik Night-Pryanik merged commit 2d4306f into CleverRaven:master Nov 16, 2024
21 of 27 checks passed
@github-actions github-actions bot added the BasicBuildPassed This PR builds correctly, label assigned by github actions label Nov 16, 2024
@AlexMooney AlexMooney deleted the use_heavy_training_dummy branch November 16, 2024 19:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions Melee Melee weapons, tactics, techniques, reach attack
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants