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

Expanding bronze: Weapons #56381

Merged
merged 6 commits into from
Apr 4, 2022
Merged

Conversation

Jarewill
Copy link
Contributor

Summary

Content "Adds more bronze weapons"

Purpose of change

Follow up on #55815
Adds three more survivor-grade bronze weapons: A sword, a spear and a mace.
The sword and spear are based on the xiphos and dory with lower damage and lower to-hit bonus.
The mace is an upgrade of the trench mace with higher damage and durability at the cost of more weight. It's still weaker than the actual mace.

Describe the solution

Added a bronze sword, which is a weaker version of the xiphos.
Added a bronze spear, which is a weaker version of the dory.
Added a bronze mace, which is between the trench mace and the mace.

Describe alternatives you've considered

Leaving bronze weapons as they are currently.
Or any tweaks to the added weapons.
I am completely open here.

Testing

Loaded the game without errors and crafted the weapons.

Additional context

Comparing the items:
image
image
image
I have no idea why lower price is green.

@github-actions github-actions bot added [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions astyled astyled PR, label is assigned by github actions labels Mar 27, 2022
@Faalagorn
Copy link
Contributor

I think lower price is green if you want to buy the item, but yeah, it's confusing. Maybe worth changing the other way around?

@github-actions github-actions bot added the BasicBuildPassed This PR builds correctly, label assigned by github actions label Mar 27, 2022
@kevingranade kevingranade merged commit 422846d into CleverRaven:master Apr 4, 2022
Drew4484 added a commit to Drew4484/Cataclysm-DDA that referenced this pull request Apr 4, 2022
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants