Arsenal tech overhaul #2104
Merged
Arsenal tech overhaul #2104
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
Signed-off-by: Stop-Signs <[email protected]>
Signed-off-by: Stop-Signs <[email protected]>
Signed-off-by: Stop-Signs <[email protected]>
Signed-off-by: Stop-Signs <[email protected]>
Signed-off-by: Stop-Signs <[email protected]>
Signed-off-by: Stop-Signs <[email protected]>
|
why did it request velcro?? no offense |
|
i have a feeling that these errors are not fake :3 |
IamVelcroboy
reviewed
Nov 4, 2024
Signed-off-by: Stop-Signs <[email protected]>
IamVelcroboy
reviewed
Nov 5, 2024
IamVelcroboy
reviewed
Nov 5, 2024
IamVelcroboy
reviewed
Nov 5, 2024
IamVelcroboy
reviewed
Nov 5, 2024
|
@IamVelcroboy for the love of all that is holy why did you submit this as 5 individual reviews |
Signed-off-by: Stop-Signs <[email protected]>
Signed-off-by: Stop-Signs <[email protected]>
Signed-off-by: Stop-Signs <[email protected]>
Signed-off-by: Stop-Signs <[email protected]>
|
I HOPE that fixed it, we shall see |
Radezolid
reviewed
Nov 5, 2024
🤷♂️ I was on my phone, looking through the file changes |
Signed-off-by: deltanedas <[email protected]>
deltanedas
previously approved these changes
Nov 6, 2024
|
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
MilonPL
suggested changes
Nov 6, 2024
Signed-off-by: Milon <[email protected]>
…/Delta-v into Arsenal-Tech-Overhaul
MilonPL
added
S: Awaiting Changes
MilonPL
suggested changes
Nov 6, 2024
Co-authored-by: Milon <[email protected]> Signed-off-by: Stop-Signs <[email protected]>
Signed-off-by: Stop-Signs <[email protected]>
Signed-off-by: Stop-Signs <[email protected]>
github-actions
bot
added
S: Needs Review
and removed
S: Awaiting Changes
Signed-off-by: Stop-Signs <[email protected]>
|
ubuntu error is fake, this is ready to merge |
MilonPL
suggested changes
Nov 7, 2024
Signed-off-by: Stop-Signs <[email protected]>
Signed-off-by: Stop-Signs <[email protected]>
MilonPL
approved these changes
Nov 7, 2024
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
Merges some of the redundant techs, reduces the amount of content locked behind getting t3 arsenal. adjusts some pricing and moves the incendiary ammo and uranium ammo techs to tier 2.
Why / Balance
The 2 laser cannons being single item techs really wasnt doing sec any favors, neither was having the pdw and ik60 cost 20k combined. merging them with a slight price increase should overall be better for the game. moving the advanced munitions to t2 is kind of just where i thought they fit better, they are t1's that both cost a large amount of money, and they aren't exactly basic things for sec. Svallins being moved to t2 shouldn't cause many issues as the gun sucks and sec can just confiscate them(if needed I can move the recipe location, remove the tech alltogether, or move it back to t3) Basic shuttle armament is now a t1, but you have to get explosive tech and salv weaponry first. advanced shuttle armament is now a t2. hopefully with events involving shuttles coming soon, this will allow sec to be able to have those oh so fun shuttle battles. Advanced laser pistol is remaining a t3 for obvious reasons.
Technical details
Comments were left on everything (i hope) still working on that new t3, will update when I do.
Media
Requirements
Changelog
🆑