Skip to content

Comments

Path of the Gun - Munitioneer Gets A Firearm Loadout Choice#251

Open
EtheoBoxxman wants to merge 1 commit intoCausticCove:mainfrom
EtheoVteran:munitions
Open

Path of the Gun - Munitioneer Gets A Firearm Loadout Choice#251
EtheoBoxxman wants to merge 1 commit intoCausticCove:mainfrom
EtheoVteran:munitions

Conversation

@EtheoBoxxman
Copy link

About The Pull Request

Title.
Gives Munitioneer a fourth loadout, bumps their firearms up to expert and gives them expert engineering so they can make firearm stuff without needing to sleep a minimum of three times, and novice arcyne so they can craft their own infernal powder for the flasks.
Like their other loadouts, it gives them everything they need to use the combat skill.

Developer's checklist

  • I have modularized whatever changes I could.
  • I have marked the start and end of my edits outside the caustic modular folder (if applicable) for changes that I couldn't modularize.
  • It compiled locally, and I tested new features, or potential issues with related features, to the best of my abilities.
  • This change applies mainly to our server, and wouldn't be in a better place upstream / cannot wait until the weekly upstream update.

Testing Evidence

Ran a debug server, everything shows up as expected.

Why It's Good For The Game

Since upstream doesnt have guns, this change is only really relevant to us. They're called munitioneer after all, so why not give them an option for firearms?
This would probably technically make it harder for munitioneer to survive combat without ally support, as they dont get armor training or dodge expert, but its a cool gimmick.

Changelog

🆑
add: Added new mechanics or gameplay changes
add: Added more things
del: Removed old things
qol: made something easier to use
balance: rebalanced something
fix: fixed a few things
sound: added/modified/removed audio or sound effects
image: added/modified/removed some icons or images
map: added/modified/removed map content
spellcheck: fixed a few typos
code: changed some code
refactor: refactored some code
config: changed some config setting
admin: messed with admin stuff
server: something server ops should know
/:cl:

@SpadesNeil
Copy link

This branch has conflicts that must be resolved

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants