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

Rubber Bullets #1388

Merged
merged 10 commits into from
Jun 1, 2024
Merged

Conversation

dvir001
Copy link
Contributor

@dvir001 dvir001 commented May 18, 2024

About the PR

Restore all rubber bullets

Why / Balance

No need to be removed

How to test

Items are back, spawn menu

Media

N/A

Breaking changes

N/A

Changelog
🆑 dvir01

  • tweak: Rubber bullets are back in production, you can ones again enjoy micro plastics in your blood!

@dvir001 dvir001 marked this pull request as ready for review May 18, 2024 21:40
@dvir001 dvir001 added S: Awaiting Changes This PR has changes that need to be made before merging S: DO NOT MERGE labels May 19, 2024
@dvir001 dvir001 changed the title Rubber Bullets (Draft) Rubber Bullets May 20, 2024
@dvir001 dvir001 added S: Needs Review This PR is awaiting reviews and removed S: Awaiting Changes This PR has changes that need to be made before merging S: DO NOT MERGE labels May 20, 2024
@dvir001 dvir001 requested review from GreaseMonk and Cheackraze May 20, 2024 23:34
@dvir001
Copy link
Contributor Author

dvir001 commented May 20, 2024

Done

Copy link
Contributor

@whatston3 whatston3 left a comment

Choose a reason for hiding this comment

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

Changes look good. Only thing I think is a bit strange is how the rubber cartridges that spawn in non-lethal revolvers are copper-jacketed instead of painted blue. Looking into that for curiosity, otherwise looks good.

I've pushed whitespace YAML cleanup onto a branch on my repo (GitHub). Requesting changes only for cleanup.

@whatston3 whatston3 mentioned this pull request May 26, 2024
1 task
@GreaseMonk GreaseMonk merged commit ad53d7c into new-frontiers-14:master Jun 1, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
No C# S: Needs Review This PR is awaiting reviews YML
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants