-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
More Gun Variants, Part Iv #74691
More Gun Variants, Part Iv #74691
Conversation
Spell checker encountered unrecognized words in the in-game text added in this pull request. See below for details. Click to expand
This alert is automatically generated. You can simply disregard if this is inaccurate, or (optionally) you can also add the new words to Hints for adding a new word to the dictionary
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you also remove these guns from this list?
Cataclysm-DDA/tools/json_tools/gun_variant_validator.py
Lines 108 to 111 in f1b4459
"m107a1", | |
"as50", | |
"tac50", | |
"bfg50", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. I made some looks like suggestions but it's your call. Let me know which way you decide and this is ready for merge.
Co-authored-by: Maleclypse <[email protected]>
Co-authored-by: Maleclypse <[email protected]>
Co-authored-by: Maleclypse <[email protected]>
Co-authored-by: Maleclypse <[email protected]>
Summary
None
Purpose of change
Continues work left off in… #62074, over 10000 PRs ago. Wow, this was a long time coming.
We have a lot of guns. We have a variant system for non-gun-savvy people. We don’t have variants for all our guns. This is not a great state of being.
Describe the solution
I have added generic variants for all the firearms in .50-calibre, as well as their magazines. I have also tweeked some of the pre-existing descriptions, mainly to set some prose in the past tense to reflect the game setting, and edited the AS-50 description to make it more concise.
Describe alternatives you've considered
Adding variants for different guns.
Testing
Additional context
Consult, for prior installments of this effort, PRs: