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

Crude weapons 2020 #39711

Merged
merged 48 commits into from
Apr 27, 2020
Merged

Conversation

xanderrootslayer
Copy link
Contributor

@xanderrootslayer xanderrootslayer commented Apr 19, 2020

Summary

SUMMARY: Content "four new tools, one renamed tool, and four crafting recipes."

Purpose of change

The recent SUS bathroom updates added some items which were otherwise junk. This PR started as an effort to give them an immediate purpose, but grew beyond that.

Describe the solution

The Sharpened Toothbrush was changed into the more general Plastic Shank, which is now craftable. Bones may be fashioned into a Bone Shiv. A new, somewhat stronger whip has been added (the Scourge), along with a mid-range medieval dagger (the Baselard); both may be crafted. The Machete Multitool was added, with all its real-life pros and cons.

Describe alternatives you've considered

The Machete Multitool was the last item I thought of. Yes, it is the Camillus Carnivore. Yes, it is that fragile in real life. I decided against adding a crafting recipe for it; a survivor already has better options by the time they could plausibly forge one.

The Baselard was added to prepare for a future PR, it could have easily been delayed.

Testing

No fatal errors!

Additional context

turns out the Sharpened Toothbrush still sucks, who knew?

data/json/recipes/recipe_weapon.json Outdated Show resolved Hide resolved
data/json/recipes/recipe_weapon.json Outdated Show resolved Hide resolved
data/json/recipes/recipe_weapon.json Outdated Show resolved Hide resolved
@Zireael07
Copy link
Contributor

Is that a mod or is it for the game itself?

some suggested edits
realized what was missing
hasty...
Copy link
Contributor

@mlangsdorf mlangsdorf left a comment

Choose a reason for hiding this comment

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

Revisions suggested to bring these in line with the rebalanced melee weapons.

data/json/items/melee/bludgeons.json Show resolved Hide resolved
data/json/items/melee/bludgeons.json Outdated Show resolved Hide resolved
data/json/items/melee/misc.json Show resolved Hide resolved
data/json/items/melee/swords_and_blades.json Show resolved Hide resolved
data/json/items/melee/swords_and_blades.json Outdated Show resolved Hide resolved
@mlangsdorf mlangsdorf added the Melee Melee weapons, tactics, techniques, reach attack label Apr 20, 2020
knives updated to match 0.E standards
never half-ass it.
forgot something
oops
kevingranade and others added 17 commits April 24, 2020 05:25
linters only detect fatal crashes, not nonsense that technically still runs.
dangit
it was genuinely just spelled wrong.
more errors, hopefully the last
appveyor enforces style too
also bad speiling
I must have branched RIGHT before the plurals were updated.
Sorry about the pikes, I worked around them
it was copied from master...
pasted wrong
perhaps I should have just copied the master
it changed while I was running validation!
why is Science even in here
@kevingranade kevingranade merged commit c87aaea into CleverRaven:master Apr 27, 2020
ZhilkinSerg pushed a commit to ZhilkinSerg/Cataclysm-DDA that referenced this pull request Apr 27, 2020
Drewscriver pushed a commit to Drewscriver/Cataclysm-DDA that referenced this pull request Apr 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Melee Melee weapons, tactics, techniques, reach attack
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants