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

Re-organizes xeno code, removes one letter vars etc #7935

Closed
wants to merge 18 commits into from

Conversation

Red-byte3D
Copy link
Contributor

@Red-byte3D Red-byte3D commented Dec 29, 2024

About the pull request

Reorganizes strain / "power" code into their own files as opposed to caste_powers.dm

Explain why it's good for the game

Should make it clearer to work on and easier to comprehend for newer people trying to contribute, some castes get too bloated like praetorian having over 1k+ lines, categorizes all abilities inot their own.dm files so you can just search for the strain / caste name to find all abilities.

Testing Photographs and Procedure

Screenshots & Videos

Put screenshots and videos here with an empty line between the screenshots and the <details> tags.

Changelog

🆑
code: Reorganizes all xeno code into the file of their own caste or strain, replaces a ton of one letter vars
/:cl:

@cm13-github
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@cm13-github cm13-github added the Merge Conflict PR can't be merged because it touched too much code label Dec 29, 2024
@cmss13-ci cmss13-ci bot added the Code Improvement Make the code longer label Dec 29, 2024
InsaneRed added 2 commits December 29, 2024 06:12
@cm13-github cm13-github removed the Merge Conflict PR can't be merged because it touched too much code label Dec 29, 2024
@cm13-github
Copy link
Contributor

Conflicts have been resolved. A maintainer will review the pull request shortly.

@cm13-github
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@cm13-github cm13-github added the Merge Conflict PR can't be merged because it touched too much code label Jan 2, 2025
@cm13-github cm13-github removed the Merge Conflict PR can't be merged because it touched too much code label Jan 3, 2025
@cm13-github
Copy link
Contributor

Conflicts have been resolved. A maintainer will review the pull request shortly.

@cm13-github cm13-github added the Merge Conflict PR can't be merged because it touched too much code label Jan 10, 2025
@cm13-github
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@Red-byte3D
Copy link
Contributor Author

willing to work on this further if a maint give sapproval

@Red-byte3D Red-byte3D closed this Jan 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code Improvement Make the code longer Merge Conflict PR can't be merged because it touched too much code
Projects
Status: Closed
Development

Successfully merging this pull request may close these issues.

2 participants