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

Bullet stacks #3456

Merged
merged 34 commits into from
Oct 12, 2024
Merged

Bullet stacks #3456

merged 34 commits into from
Oct 12, 2024

Conversation

thgvr
Copy link
Member

@thgvr thgvr commented Sep 29, 2024

About The Pull Request

Partial port of Mojave-Sun/mojave-sun-13#2076 and surrounding PRs
Adds bullet handfuls. You can only have the same type of ammunition in a handful. No mismatched stacks
Spent casings are not stackable.

Generally, the rule is:

  • By default, stacks can contain 12 rounds.
  • Pistol-equivalent stacks contain 15 rounds.
  • Shotshell stacks contain 8 rounds.
  • Rifle stacks contain 15 rounds.
  • Large calibre stacks contain 5 rounds.

Also replaces old ammo boxes with storage boxes that come pre-filled with stacks of bullets like this:
image
Here's how the stacks look:
image
image

Changelog

🆑
add: Added bullet stacks, they allow you to stack ammo of the same type into a group.
balance: Changed old ammo boxes into storage items containing stacks of bullets
/:cl:

@github-actions github-actions bot added DME Edit Code change Watch something violently break. labels Sep 29, 2024
@thgvr thgvr marked this pull request as draft September 29, 2024 08:10
@Sun-Soaked
Copy link
Member

it's peam

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

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

@github-actions github-actions bot added the Sprites A bikeshed full of soulless bikes. label Sep 29, 2024
@github-actions github-actions bot added Map Change Tile placing is hard. Thank you for your service. and removed Merge Conflict Use Git Hooks, you're welcome. labels Oct 5, 2024
@github-actions github-actions bot added the Merge Conflict Use Git Hooks, you're welcome. label Oct 5, 2024
Copy link
Contributor

github-actions bot commented Oct 5, 2024

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

@github-actions github-actions bot removed the Map Change Tile placing is hard. Thank you for your service. label Oct 5, 2024
@github-actions github-actions bot removed the Merge Conflict Use Git Hooks, you're welcome. label Oct 5, 2024
@github-actions github-actions bot added the Map Change Tile placing is hard. Thank you for your service. label Oct 5, 2024
@thgvr thgvr changed the title Bullet handfuls Bullet stacks Oct 5, 2024
@generalthrax
Copy link
Contributor

You can get invisible 0 bullet stacks. Just activate 1 bullet bullet stacks in your hand.

@Sun-Soaked
Copy link
Member

image that's a bug

my cranial implant explodes
I should just do a regex find on all these

Copy link
Contributor

github-actions bot commented Oct 8, 2024

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

@github-actions github-actions bot added the Merge Conflict Use Git Hooks, you're welcome. label Oct 8, 2024
@github-actions github-actions bot removed the Merge Conflict Use Git Hooks, you're welcome. label Oct 8, 2024
@thgvr thgvr marked this pull request as ready for review October 10, 2024 01:59
@thgvr thgvr requested review from a team as code owners October 10, 2024 01:59
@thgvr
Copy link
Member Author

thgvr commented Oct 10, 2024

This is now ready

Comment on lines 5 to 10
desc = "A 7-round speed loader for quickly reloading .357 revolvers. These rounds do good damage with average performance against armor."
icon_state = "speedloader_357-6"
base_icon_state = "speedloader_357"
ammo_type = /obj/item/ammo_casing/a357
caliber = ".357"
max_ammo = 7
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
desc = "A 7-round speed loader for quickly reloading .357 revolvers. These rounds do good damage with average performance against armor."
icon_state = "speedloader_357-6"
base_icon_state = "speedloader_357"
ammo_type = /obj/item/ammo_casing/a357
caliber = ".357"
max_ammo = 7
desc = "A 6-round speed loader for quickly reloading .357 revolvers. These rounds do good damage with average performance against armor."
icon_state = "speedloader_357-6"
base_icon_state = "speedloader_357"
ammo_type = /obj/item/ammo_casing/a357
caliber = ".357"
max_ammo = 6

@rye-rice rye-rice added this pull request to the merge queue Oct 12, 2024
Merged via the queue into shiptest-ss13:master with commit 7e6d3dd Oct 12, 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
Partial port of Mojave-Sun/mojave-sun-13#2076
and surrounding PRs
Adds bullet handfuls. You can only have the same type of ammunition in a
handful. No mismatched stacks
Spent casings are not stackable.

Generally, the rule is:
* By default, stacks can contain 12 rounds.
* Pistol-equivalent stacks contain 15 rounds.
* Shotshell stacks contain 8 rounds.
* Rifle stacks contain 15 rounds.
* Large calibre stacks contain 5 rounds.

Also replaces old ammo boxes with storage boxes that come pre-filled
with stacks of bullets like this:

![image](https://github.com/user-attachments/assets/cb71a690-cbd2-4973-bfca-4e8b9f472349)
Here's how the stacks look:

![image](https://github.com/user-attachments/assets/8cb64f4e-8b4c-4b87-9c60-96a8354a69de)

![image](https://github.com/user-attachments/assets/2b42e546-1d04-4542-b6dd-19c9099991e2)

## Changelog

:cl:
add: Added bullet stacks, they allow you to stack ammo of the same type
into a group.
balance: Changed old ammo boxes into storage items containing stacks of
bullets
/: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: thgvr <[email protected]>
zimon9 pushed a commit to zimon9/Shiptest-PR-testing that referenced this pull request Oct 29, 2024
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. Sprites A bikeshed full of soulless bikes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants