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

Serene Sporting #3604

Merged
merged 17 commits into from
Oct 26, 2024
Merged

Serene Sporting #3604

merged 17 commits into from
Oct 26, 2024

Conversation

Erikafox
Copy link
Contributor

@Erikafox Erikafox commented Oct 20, 2024

About The Pull Request

Adds new guns sprited by @rye-rice
M17 pistol - .22lr - 10rd mag
M12 rifle - .22lr - 25rd mag
m13 rifle - .22lr - 25rd mag - 3rd burst
m15 rifle - 5.56 clip - 20rd mag
m11 shotgun - 12g - 8rd tube

dreamseeker_55G0WFwm5r

Watch the video here

Why It's Good For The Game

project content into the world

Changelog

🆑 Apogeesys, Ryerice, Erikafox
add: Serene Outdoors has begun a new production run of its classics - the Model 17, Model 12, Model 15, and Model 11
del: old combat shotgun
/:cl:

@Erikafox Erikafox requested review from FalloutFalcon and a team as code owners October 20, 2024 20:47
@github-actions github-actions bot added Map Change Tile placing is hard. Thank you for your service. DME Edit Sprites A bikeshed full of soulless bikes. Code change Watch something violently break. labels Oct 20, 2024
@Apogee-dev
Copy link
Contributor

Would you mind putting an image of the sprites in the PR body? just a screenshot of the project file if need be, it's just that the resolution on the video isn't perfect and factors like lighting can make things hard to see.

@Erikafox
Copy link
Contributor Author

done

Copy link
Contributor

@Apogee-dev Apogee-dev left a comment

Choose a reason for hiding this comment

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

Mostly just some proofreading, there were a couple grammar errors on the descs and a note on the super sporter crate that's entirely optional.

code/modules/cargo/packs/gun.dm Outdated Show resolved Hide resolved
code/modules/cargo/packs/gun.dm Outdated Show resolved Hide resolved
code/modules/cargo/packs/gun.dm Outdated Show resolved Hide resolved
@MemeSnorfer
Copy link
Contributor

One major complaint thing I wanted to point out; The sporter and super sporter hunting rifles would apply penalties to characters with the gunslinger quirk, since these rifles are technically automatic weapons with a burst size of 1.

@Apogee-dev
Copy link
Contributor

Apogee-dev commented Oct 21, 2024

One major complaint thing I wanted to point out; The sporter and super sporter hunting rifles would apply penalties to characters with the gunslinger quirk, since these rifles are technically automatic weapons with a burst size of 1.

Isn't this already the case with other semi-automatic weapons? I thought gunslinger also applied penalties to other self-loaders like the Candor too.
EDIT: It doesn't, I misread. Gunslinger penalties are applied to assault rifles, LMGs, and SMGs as classes. Technically it doesn't even apply to machine pistols. This ALSO means it doesn't actually apply to the Sporter or Super Sporter either, because they're just under the base ballistic/automatic type instead of the smg or assault subtypes. So yeah, none of these guns get gunslinger penalties or bonuses (except for the Buckmaster, which gets -1 gunslinger recoil like all shotguns). You might want to slap a gunslinger penalty on the Larker, but otherwise it can be ignored.

@MemeSnorfer
Copy link
Contributor

MemeSnorfer commented Oct 21, 2024

One major complaint thing I wanted to point out; The sporter and super sporter hunting rifles would apply penalties to characters with the gunslinger quirk, since these rifles are technically automatic weapons with a burst size of 1.

Isn't this already the case with other semi-automatic weapons? I thought gunslinger also applied penalties to other self-loaders like the Candor too. EDIT: It doesn't, I misread. Gunslinger penalties are applied to assault rifles, LMGs, and SMGs as classes. Technically it doesn't even apply to machine pistols. This ALSO means it doesn't actually apply to the Sporter or Super Sporter either, because they're just under the base ballistic/automatic type instead of the smg or assault subtypes.

Huh. I misremembered gunslinger's subtype penalties, then (I thought it was every weapon typed under automatic).

The machine pistols being unaffected seems like a bit of an oversight though :P

@Erikafox
Copy link
Contributor Author

One major complaint thing I wanted to point out; The sporter and super sporter hunting rifles would apply penalties to characters with the gunslinger quirk, since these rifles are technically automatic weapons with a burst size of 1.

Isn't this already the case with other semi-automatic weapons? I thought gunslinger also applied penalties to other self-loaders like the Candor too. EDIT: It doesn't, I misread. Gunslinger penalties are applied to assault rifles, LMGs, and SMGs as classes. Technically it doesn't even apply to machine pistols. This ALSO means it doesn't actually apply to the Sporter or Super Sporter either, because they're just under the base ballistic/automatic type instead of the smg or assault subtypes. So yeah, none of these guns get gunslinger penalties or bonuses (except for the Buckmaster, which gets -1 gunslinger recoil like all shotguns). You might want to slap a gunslinger penalty on the Larker, but otherwise it can be ignored.

tbh I'm of the opinion that we should remove gunslinger. It's too far into the side of gameplay altering quirk

@FalloutFalcon
Copy link
Member

One major complaint thing I wanted to point out; The sporter and super sporter hunting rifles would apply penalties to characters with the gunslinger quirk, since these rifles are technically automatic weapons with a burst size of 1.

Isn't this already the case with other semi-automatic weapons? I thought gunslinger also applied penalties to other self-loaders like the Candor too. EDIT: It doesn't, I misread. Gunslinger penalties are applied to assault rifles, LMGs, and SMGs as classes. Technically it doesn't even apply to machine pistols. This ALSO means it doesn't actually apply to the Sporter or Super Sporter either, because they're just under the base ballistic/automatic type instead of the smg or assault subtypes. So yeah, none of these guns get gunslinger penalties or bonuses (except for the Buckmaster, which gets -1 gunslinger recoil like all shotguns). You might want to slap a gunslinger penalty on the Larker, but otherwise it can be ignored.

tbh I'm of the opinion that we should remove gunslinger. It's too far into the side of gameplay altering quirk

I think it could probably stay but it should be a positive quirk that adds the few cool tricks like fanning and minor visuals like spin-cocking. Remove all the number tweaking cause its only really for powergaming

Copy link
Contributor

@Apogee-dev Apogee-dev left a comment

Choose a reason for hiding this comment

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

Oops, one last desc error. Sorry I missed that one.

code/modules/cargo/packs/gun.dm Outdated Show resolved Hide resolved
@github-actions github-actions bot added the Merge Conflict Use Git Hooks, you're welcome. label Oct 22, 2024
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot removed the Merge Conflict Use Git Hooks, you're welcome. label Oct 22, 2024
@Erikafox
Copy link
Contributor Author

nothing it just edits ships okay anyways - They're going to be in my next code PR - im gonna make a new category for magazines and check that you can purchase like - all of them

@thearbiber
Copy link

Super sporters medium sized. Not sure thats intended since its a rifle

@generalthrax
Copy link
Contributor

The M17 is bulky, too

@Erikafox
Copy link
Contributor Author

m17 should be fixed in latest. im fine with the m12/m15 being medium atm

@generalthrax
Copy link
Contributor

I unno. I just noticed the M12 being bulky while the M15 was normal. Seemed mixed up

@Erikafox
Copy link
Contributor Author

Ah. actual weights instead of 'weapon' weights

@thgvr
Copy link
Member

thgvr commented Oct 23, 2024

All these but the pistol should be unable to fit in a bag, probably.

@Erikafox
Copy link
Contributor Author

weapon weight and actual item weight are two different systems.
Weapon weight is "can you fire while double-wielding/without wielding"
item weight is you know what it is.

either way fixed

@github-actions github-actions bot added the Merge Conflict Use Git Hooks, you're welcome. label Oct 24, 2024
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot removed the Merge Conflict Use Git Hooks, you're welcome. label Oct 24, 2024
@github-actions github-actions bot added the Merge Conflict Use Git Hooks, you're welcome. label Oct 25, 2024
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

Signed-off-by: Erika Fox <94164348+Erikafox@users.noreply.github.com>
@github-actions github-actions bot removed the Merge Conflict Use Git Hooks, you're welcome. label Oct 25, 2024
@rye-rice rye-rice removed the request for review from FalloutFalcon October 26, 2024 01:43
Copy link
Member

@Sun-Soaked Sun-Soaked left a comment

Choose a reason for hiding this comment

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

looks good

/obj/item/gun/ballistic/shotgun/automatic/m11/Initialize()
. = ..()
if(prob(1))
name = "Model 11 \"Cuckmaster\""
Copy link
Member

Choose a reason for hiding this comment

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

image

Copy link
Member

Choose a reason for hiding this comment

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

remove this, please.

@Sun-Soaked Sun-Soaked added this pull request to the merge queue Oct 26, 2024
Merged via the queue into shiptest-ss13:master with commit cb4768b Oct 26, 2024
16 checks passed
zimon9 pushed a commit to zimon9/Shiptest-PR-testing that referenced this pull request Oct 29, 2024
<!-- Write **BELOW** The Headers and **ABOVE** The comments else it may
not be viewable. -->
<!-- You can view Contributing.MD for a detailed description of the pull
request process. -->

## About The Pull Request
Adds new guns sprited by @rye-rice 
M17 pistol - .22lr - 10rd mag
M12 rifle - .22lr - 25rd mag
m13 rifle - .22lr - 25rd mag - 3rd burst
m15 rifle - 5.56 clip - 20rd mag
m11 shotgun - 12g - 8rd tube


![dreamseeker_55G0WFwm5r](https://github.com/user-attachments/assets/b9d9f91e-6699-4f40-a5e5-163d70069691)


[Watch the video here](https://youtu.be/O1TrkZ_FHFU?si=-dAk-dlTSsug8X0R)

<!-- Describe The Pull Request. Please be sure every change is
documented or this can delay review and even discourage maintainers from
merging your PR! -->

## Why It's Good For The Game
project content into the world
<!-- Please add a short description of why you think these changes would
benefit the game. If you can't justify it in words, it might not be
worth adding. -->

## Changelog

:cl: Apogeesys, Ryerice, Erikafox 
add: Serene Outdoors has begun a new production run of its classics -
the Model 17, Model 12, Model 15, and Model 11
del: old combat shotgun
/:cl:

<!-- Both :cl:'s are required for the changelog to work! You can put
your name to the right of the first :cl: if you want to overwrite your
GitHub username as author ingame. -->
<!-- You can use multiple of the same prefix (they're only used for the
icon ingame) and delete the unneeded ones. Despite some of the tags,
changelogs should generally represent how a player might be affected by
the changes rather than a summary of the PR's contents. -->

---------

Signed-off-by: Erika Fox <94164348+Erikafox@users.noreply.github.com>
Co-authored-by: Erika Fox <94164348+Bjarl@users.noreply.github.com>
zimon9 pushed a commit to zimon9/Shiptest-PR-testing that referenced this pull request Oct 29, 2024
MrCat15352 pushed a commit to MrCat15352/MrCat that referenced this pull request Dec 27, 2024
<!-- Write **BELOW** The Headers and **ABOVE** The comments else it may
not be viewable. -->
<!-- You can view Contributing.MD for a detailed description of the pull
request process. -->

## About The Pull Request
Adds new guns sprited by @rye-rice 
M17 pistol - .22lr - 10rd mag
M12 rifle - .22lr - 25rd mag
m13 rifle - .22lr - 25rd mag - 3rd burst
m15 rifle - 5.56 clip - 20rd mag
m11 shotgun - 12g - 8rd tube


![dreamseeker_55G0WFwm5r](https://github.com/user-attachments/assets/b9d9f91e-6699-4f40-a5e5-163d70069691)


[Watch the video here](https://youtu.be/O1TrkZ_FHFU?si=-dAk-dlTSsug8X0R)

<!-- Describe The Pull Request. Please be sure every change is
documented or this can delay review and even discourage maintainers from
merging your PR! -->

## Why It's Good For The Game
project content into the world
<!-- Please add a short description of why you think these changes would
benefit the game. If you can't justify it in words, it might not be
worth adding. -->

## Changelog

:cl: Apogeesys, Ryerice, Erikafox 
add: Serene Outdoors has begun a new production run of its classics -
the Model 17, Model 12, Model 15, and Model 11
del: old combat shotgun
/:cl:

<!-- Both :cl:'s are required for the changelog to work! You can put
your name to the right of the first :cl: if you want to overwrite your
GitHub username as author ingame. -->
<!-- You can use multiple of the same prefix (they're only used for the
icon ingame) and delete the unneeded ones. Despite some of the tags,
changelogs should generally represent how a player might be affected by
the changes rather than a summary of the PR's contents. -->

---------

Signed-off-by: Erika Fox <94164348+Erikafox@users.noreply.github.com>
Co-authored-by: Erika Fox <94164348+Bjarl@users.noreply.github.com>
MrCat15352 pushed a commit to MrCat15352/MrCat that referenced this pull request Jan 3, 2025
<!-- Write **BELOW** The Headers and **ABOVE** The comments else it may
not be viewable. -->
<!-- You can view Contributing.MD for a detailed description of the pull
request process. -->
Adds new guns sprited by @rye-rice
M17 pistol - .22lr - 10rd mag
M12 rifle - .22lr - 25rd mag
m13 rifle - .22lr - 25rd mag - 3rd burst
m15 rifle - 5.56 clip - 20rd mag
m11 shotgun - 12g - 8rd tube

![dreamseeker_55G0WFwm5r](https://github.com/user-attachments/assets/b9d9f91e-6699-4f40-a5e5-163d70069691)

[Watch the video here](https://youtu.be/O1TrkZ_FHFU?si=-dAk-dlTSsug8X0R)

<!-- Describe The Pull Request. Please be sure every change is
documented or this can delay review and even discourage maintainers from
merging your PR! -->
project content into the world
<!-- Please add a short description of why you think these changes would
benefit the game. If you can't justify it in words, it might not be
worth adding. -->

:cl: Apogeesys, Ryerice, Erikafox
add: Serene Outdoors has begun a new production run of its classics -
the Model 17, Model 12, Model 15, and Model 11
del: old combat shotgun
/:cl:

<!-- Both :cl:'s are required for the changelog to work! You can put
your name to the right of the first :cl: if you want to overwrite your
GitHub username as author ingame. -->
<!-- You can use multiple of the same prefix (they're only used for the
icon ingame) and delete the unneeded ones. Despite some of the tags,
changelogs should generally represent how a player might be affected by
the changes rather than a summary of the PR's contents. -->

---------

Signed-off-by: Erika Fox <94164348+Erikafox@users.noreply.github.com>
Co-authored-by: Erika Fox <94164348+Bjarl@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code change Watch something violently break. DME Edit Map Change Tile placing is hard. Thank you for your service. Ruin Ruin successfully tested. Ship Ship successfully tested. Sprites A bikeshed full of soulless bikes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants