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

Adds Chisel ship. #504

Merged
merged 6 commits into from
Oct 29, 2023
Merged

Adds Chisel ship. #504

merged 6 commits into from
Oct 29, 2023

Conversation

Blackszedows
Copy link
Contributor

@Blackszedows Blackszedows commented Oct 28, 2023

About the PR

Added a reclaiming vessel , Chisel

Why / Balance

I wanted to make a ship meant for scrapping abandoned ships in case the ship's captain's ID wasn't aviable, however after concluding that using shuttle apu's wouldn't be balanced and using PACMAN's instead , I had to give the ship some form of mining capabilities.

Media

2023-10-29_21 06 51

  • I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog
🆑

  • add: Added Chisel shuttle

@terezi-station
Copy link
Contributor

great start to a better prospector, good shape and layout but needs work on the interior and walls, non connected diagonals dont exist on any other ship or station

@AjexRose
Copy link
Contributor

maybe add a nitrogen canister along with the oxygen canister, or replace it with a general air canister for slimes.

Removed YouTool vendor
Reclaimer ship actually has a reclaimer onboard now
@Cheackraze
Copy link
Member

this is actually a super cute prospector-like 'variant', or simple upgrade. I like it. My only critique is the triple thick wall chunk in the lower right. I have an idea about that actually... how about a secret door for a smuggling compartment back there! Maybe move the generator or locker over and put a secret door would be friggen cool

@Blackszedows
Copy link
Contributor Author

Judging by the seemingly unnecessary commits it adds, in the possible future PR's i'll try not to sync branches mid-review.

@Cheackraze
Copy link
Member

Judging by the seemingly unnecessary commits it adds, in the possible future PR's i'll try not to sync branches mid-review.

we do squash commits when merging, meaning it takes all of your changes and rolls them up into 1 commit, so dont worry about the # of commits you have on your branch

@Cheackraze Cheackraze merged commit 47d962d into new-frontiers-14:master Oct 29, 2023
8 of 9 checks passed
FrontierATC added a commit that referenced this pull request Oct 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants