OVERHAULED the Ads Small Ships and REDESIGNED Them#3269
OVERHAULED the Ads Small Ships and REDESIGNED Them#3269EldritchPineapple wants to merge 16 commits intoMonolith-Station:mainfrom
Conversation
|
welcome home glimmer from starsector |
There was a problem hiding this comment.
Put actual effort into your PR body and Why / Balance or your PR may be denied off that basis alone.
Fix your Linter and test fails.
You will need to implement the shuttle to actually be spawned as an event.
See
Resources\Prototypes\_Mono\GameRules\Shuttles\ship_spawns.yml
Resources\Prototypes\_Mono\GameRules\damaged_ai.yml
Resources/Prototypes/_Mono/GameRules/base.yml
You will also need to make the shuttle properly labeled so it shows up as "ADS Whatever" or the like and have a blue outline, just look at how other ADS ships do it or whatever. In case you haven't already done this.
Direct and concise to what is added and not a meme. "Added ADS X and ADS X"
You also have not removed the Razor at all. So that's inaccurate.
… added lights to it too
|
review needed |
|
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
2a1f1ca to
f15d0e7
Compare
…e/Monolith into ads-small-ships
|
conficts resolved!! |
UnicornOnLSD
left a comment
There was a problem hiding this comment.
Show me the razor to see improved replacements
There was a problem hiding this comment.
Fencer & Lancer(n)
Adjust to meet ADS guidelines
remove IFF console
Need path to the outside (external airlock not docking)
Adjust to meet thruster guidelines
I'll allow the gunneries to stay as high ones for gamer loot.
Shapes & weapons looking good otherwise
Lancern goes boom
An upgrade to the razor, otherwise. Good work.

About the PR
removed the razor and added three new ads small ships (one is evil and secret)
Why / Balance
starch ship and outdated, also fits ads guidelines more
How to test
build all release and test with the different gamerules
Media
Requirements
Breaking changes
naur
Changelog
🆑