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

Add longest_side to bullets #65184

Merged
merged 5 commits into from
Apr 21, 2023

Conversation

GuardianDll
Copy link
Member

Summary

None

Purpose of change

Doing some stuff, i found a need to calculate the length of the bullets, and found the game has lack of bullet length for almost all rounds. As i will use this info anyway, i decide to fix it also, as it may cause some issues, when charges went away

Describe the solution

Simply add longest_side to most bullet in the game - only 1 change per file, as bullets copy-from each other
The omitted bullet lengths are paper cartridges, some heavy stuff (as they have it already) and caseless rounds

@github-actions github-actions bot added the [JSON] Changes (can be) made in JSON label Apr 19, 2023
@GuardianDll GuardianDll changed the title initial commit Add longest_side to bullets Apr 19, 2023
@github-actions github-actions bot added json-styled JSON lint passed, label assigned by github actions astyled astyled PR, label is assigned by github actions labels Apr 19, 2023
@github-actions github-actions bot removed the json-styled JSON lint passed, label assigned by github actions label Apr 19, 2023
@github-actions github-actions bot added the json-styled JSON lint passed, label assigned by github actions label Apr 19, 2023
@github-actions github-actions bot added Items: Armor / Clothing Armor and clothing BasicBuildPassed This PR builds correctly, label assigned by github actions labels Apr 21, 2023
@bombasticSlacks bombasticSlacks merged commit 2662790 into CleverRaven:master Apr 21, 2023
@GuardianDll GuardianDll deleted the bullet_length branch April 21, 2023 18:44
katemonster33 pushed a commit to katemonster33/Cataclysm-DDA that referenced this pull request May 17, 2023
* initial commit

* Add length to 50 bmg ammo box, so 50 bmg can fit here

* Update data/json/items/containers/military.json

* Update military.json

* Please work please work please work
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 Items: Armor / Clothing Armor and clothing [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.

2 participants