-
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
Gun Naming Standardization #74787
base: master
Are you sure you want to change the base?
Gun Naming Standardization #74787
Conversation
That documentation is for when the gun brand names are off. You're changing the name for when gun brand names are on |
Oh wait, that's right. But all other variants are named according to the same rules. If we should rename it back, then we need to do this for every variant. This PR just brings names to the general style for now. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. Please do not bump or comment on this issue unless you are actively working on it. Stale issues, and stale issues that are closed are still considered. |
Okay, I did it differently now. Small things: |
Why remove the variants for the Luty and 357 revolvers? |
All other Luty for different calibers also have no brand names. Probably because it is a makeshift firearm. Cataclysm-DDA/data/json/items/gun/9mm.json Lines 644 to 645 in 293f801
Because I moved the brand name to the general one. But sure I can return them back if necessary. |
It's still called a "luty", the non-brand name is to make it clear it accepts MAC-10 magazines
Isn't this defeating the purpose of the non-brand names option? |
cf64ec8
to
680b037
Compare
The .357 revolver variants are back, with one of them being slightly renamed instead. |
250efca
to
6ecb514
Compare
ad0492c
to
e037fab
Compare
Fixed a couple of things and updated names in |
i don't think removing the type identifiers from any of the weapons improves anything. someone can want to see brand names and still not have all of the weapon types memorized. |
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.
Auto-requesting reviews from non-collaborators: @bombasticSlacks
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.
Auto-requesting reviews from non-collaborators: @jbytheway
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
|
Summary
None
Purpose of change
Rename new guns to match documentation.
Describe the solution
#74787 (comment)
Describe alternatives you've considered
Testing
The game loads correctly.
Additional context
Some (actually most of them) general names are not general enough and still include the full name of the weapon.