Skip to content
This repository has been archived by the owner on Jul 15, 2022. It is now read-only.

Kapu Limbs Port (WIP) #9

Closed
wants to merge 14 commits into from
Closed

Kapu Limbs Port (WIP) #9

wants to merge 14 commits into from

Conversation

OctusGit
Copy link
Member

About The Pull Request

Original PR: BeeStation/BeeStation-Hornet#5611

Ports over Kapulimbs from Beestation in order to support the new xeno species we plan to add in the future, its much easier to use and a lot simpler in terms of optimization when it comes to handling multiple species.

Why It's Good For The Game

Multiple xeno species is going to need a better framework in order to work properly, and possibly less of a headache in the future.

Changelog

🆑
add: Kapu limbs xeno species support
/:cl:

@OctusGit OctusGit added Modular Yes I hate modularisation too, but sometimes we have to make concessions. Help wanted Extra attention is needed, we need some better coders around here. Do not merge A big red label for big (or small) mistakes. Or we're just not ready for the commitment yet. Big PR It's not the size that matters... but someone will have to review this. labels Mar 15, 2022
@OctusGit OctusGit removed the Do not merge A big red label for big (or small) mistakes. Or we're just not ready for the commitment yet. label Mar 16, 2022
Copy link
Contributor

@SabreML SabreML left a comment

Choose a reason for hiding this comment

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

Very rough initial review on some of the simpler stuff, taking it down from '100 errors, 8 warnings' to '16 errors, 1 warning'.
image
image

(As a disclaimer, the code still isn't compiling yet so I can't actually test anything that I suggested, and some things may be incorrect. Either way though this gets rid of a lot of the errors.)

code/modules/mob/living/carbon/human/examine.dm Outdated Show resolved Hide resolved
code/modules/mob/living/carbon/human/species.dm Outdated Show resolved Hide resolved
code/modules/surgery/bodyparts/helpers.dm Outdated Show resolved Hide resolved
code/modules/surgery/prosthetic_replacement.dm Outdated Show resolved Hide resolved
code/modules/surgery/prosthetic_replacement.dm Outdated Show resolved Hide resolved
code/modules/surgery/prosthetic_replacement.dm Outdated Show resolved Hide resolved
code/modules/surgery/bodyparts/_bodyparts.dm Outdated Show resolved Hide resolved
Copy link
Contributor

@SabreML SabreML left a comment

Choose a reason for hiding this comment

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

A couple of things I forgot to add to the last review.

code/game/objects/items/devices/scanners.dm Outdated Show resolved Hide resolved
code/modules/mob/living/carbon/human/examine.dm Outdated Show resolved Hide resolved
code/modules/surgery/bodyparts/_bodyparts.dm Outdated Show resolved Hide resolved
code/modules/surgery/bodyparts/_bodyparts.dm Outdated Show resolved Hide resolved
code/modules/surgery/bodyparts/_bodyparts.dm Outdated Show resolved Hide resolved
code/modules/surgery/bodyparts/_bodyparts.dm Outdated Show resolved Hide resolved
code/modules/surgery/bodyparts/_bodyparts.dm Outdated Show resolved Hide resolved
OctusGit and others added 4 commits March 16, 2022 19:38
Co-authored-by: SabreML <57483089+SabreML@users.noreply.github.com>
Co-authored-by: SabreML <57483089+SabreML@users.noreply.github.com>
@OctusGit
Copy link
Member Author

tgstation/tgstation#65523
Being ported by creator themselves.

@ElorgRHG ElorgRHG added the Do not merge A big red label for big (or small) mistakes. Or we're just not ready for the commitment yet. label Mar 17, 2022
#define NOAUGMENTS 21
///will be assigned a universal vampire themed last name shared by their department. this is preferenced!
#define BLOOD_CLANS 22
//PARIAH MODULAR EDIT START
Copy link
Contributor

Choose a reason for hiding this comment

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

PSA

This isn't modular. Modular would be putting it in its own folder. // PARIAH EDIT START would make much more sense here.

Copy link
Member Author

Choose a reason for hiding this comment

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

Ah, I see I see. Thanks for the tip, I'm not that good at this stuff as you can tell lmao

@github-actions github-actions bot added the Merge Conflict Conflicts with the existing version of the codebase. label Mar 19, 2022
@ElorgRHG
Copy link
Contributor

Since upstream is doing exactly this but better, the best choice is just to take this one behind the shed I feel

@ElorgRHG ElorgRHG closed this Mar 19, 2022
@OctusGit
Copy link
Member Author

I have the slug rounds ready.

@OctusGit OctusGit deleted the kapulimbs branch March 19, 2022 23:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Big PR It's not the size that matters... but someone will have to review this. Do not merge A big red label for big (or small) mistakes. Or we're just not ready for the commitment yet. Help wanted Extra attention is needed, we need some better coders around here. Merge Conflict Conflicts with the existing version of the codebase. Modular Yes I hate modularisation too, but sometimes we have to make concessions.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants