Mosin Rework (#1391) Port over From Frontier #1148
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Testing? Mosin rework * Update snipers.yml * Update 2 * update 3 * Update snipers.yml * Update snipers.yml * Dumb updates updates do be dumb sometimes * WOULD THIS WORK? FUCK FUCK FU
SimpleStation14
changed the title
|
I just remember something, the PR for the code changes itself to allow it was in this pr not in the mosin PR |
FoxxoTrystan
requested review from
a team,
VMSolidus,
Aidenkrz,
FoxxoTrystan,
Remuchi,
DEATHB4DEFEAT,
Peptide90 and
Pspritechologist
and removed request for
a team
FoxxoTrystan
added
Priority: 4-Low
If your PR requires another PR to work properly(Such as the bugfixes needed there), you're going to need to cherrypick it too in this. |
|
I will do when I'm not in the hospital |
|
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
About the PR
This is a port over from another server frontier to allow guns to have melee attacks with it
new-frontiers-14/frontier-station-14#1391
Description
Rework the Mosin to allow Melee now, Butcher and Throwing
Why / Balance
Mosin is regarded as one of the worst weapons in the game, to the point that nearly no one uses it. Countless times, people have asked/requested to remove the bayonet from the Mosin sprite or give it the ability to be used as a melee weapon. Because Upstream changed/removed the coding for melee/range weapons, it is a bit funky, but I feel like it isn't that bad to the point where I would say it's terrible.
Attack rate of .85
- Piercing 10 (you are stabbing someone so it
- Slash 5 (you are pulling the bayonet out so it causes bleeding
How to test
Spawn in the Mosin. Shooting the rifle will now be Right click, where a "pinpoint" melee attack is left click however when the Ammo for the mosin runs dry the right click turns into a thrust attack
Media
- [X] this PR does not require an in-game showcase
Changelog
Turns out the Bayonet had a sheath on it
🆑