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

Firearms Upkeep #1421

Merged
merged 6 commits into from
Jan 22, 2025
Merged

Firearms Upkeep #1421

merged 6 commits into from
Jan 22, 2025

Conversation

DVDPlayerOfDiscordFame
Copy link

Nothing fancy here. Adding #imp to the changes I made to the vanilla firearms/magazines/ammunition, as well as any additions I've made to default arsenal research items and lathe recipe lists. Included is a description change for the Captain's Laser, as well as a fix for the Mosin's speedloader to use the correct ammunition.

🆑

  • fix: The mosin's speedloader now uses the correct ammunition.

adding #imp to the changes I made to the vanilla firearms, additions to the secfab, and research items. Included is a description change for the Captain's Laser, as well as a fix for the Mosin's speedloader to use the correct ammunition.
@kipdotnet kipdotnet self-assigned this Jan 19, 2025
Copy link

@kipdotnet kipdotnet left a comment

Choose a reason for hiding this comment

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

Works fine, although im not entirely sure why the speedloader exists? The full ammo box can load straight into the rifle just as fast and takes the same pocket space.

I moved the #imp comments I wasn't confident wouldn't show up in the examine description back to the description and tested. None of the vanilla weapons have #imp in their descriptions and the speedloader fix works.
Requested edits.
Wizden doesn't proofread and I'm fixing consistency with weapon name capitalizations. Probably not all of them but a lot of them are done here. Also fixing my own weapon name capitalizations. The fake Golden Zipper's name being all lowercase still is intentional.
Copy link

@hivehum hivehum left a comment

Choose a reason for hiding this comment

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

lgtm

@hivehum hivehum merged commit 2e41f71 into impstation:master Jan 22, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants