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

Removal of funny ammo and shuttle gun boards #2683

Merged
merged 7 commits into from
Jan 18, 2025

Conversation

blackknight954
Copy link
Contributor

@blackknight954 blackknight954 commented Jan 11, 2025

Goodbye Research

About the PR

Removes incendiary and uranium ammo and the shuttle gun boards from research. The gun boards will be added to the NFSD uplink soonTM

Why / Balance

Maintainer request

How to test

Load a server
Check the guidebook
You will notice a lack of items that were there previously

Media

image
Note the lack of Duster board

Requirements

Breaking changes

Angry playerbase?

Changelog

🆑

  • tweak: Removed Uranium and Incendiary ammo, Perforator, Svalinn machine gun, Duster, and Friendship research from the Arsenal tree

@dvir001 dvir001 self-requested a review January 12, 2025 13:19
@github-actions github-actions bot added the S: Needs Review This PR is awaiting reviews label Jan 12, 2025
@dvir001 dvir001 mentioned this pull request Jan 17, 2025
2 tasks
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 S: Merge Conflict This PR has conflicts that prevent merging label Jan 17, 2025
@github-actions github-actions bot removed the S: Merge Conflict This PR has conflicts that prevent merging label Jan 17, 2025
Copy link
Contributor

@dvir001 dvir001 left a comment

Choose a reason for hiding this comment

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

Cleanup is 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.

Looks good to me.

@whatston3 whatston3 merged commit 4f0a13a into new-frontiers-14:master Jan 18, 2025
13 checks passed
FrontierATC added a commit that referenced this pull request Jan 18, 2025
@blackknight954 blackknight954 deleted the GoodbyeResearch branch January 18, 2025 01:41
@blackknight954 blackknight954 restored the GoodbyeResearch branch January 18, 2025 01:41
@blackknight954 blackknight954 deleted the GoodbyeResearch branch January 18, 2025 01:42
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