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

Removes SSJob (Brutally) #2127

Conversation

MarkSuckerberg
Copy link
Member

About The Pull Request

SSJob is a vestigial part of the codebase, quickly bandaged to support our system. I've slowly stripped it away, but I left it in because it was rooted deeply in a few pieces of code. Today I finally just decided to rip that out too, since SSjob is continuing to break things to this day.

Removes:

  • SSjob, replacing it with three global lists
  • Job datum slot counts
  • Gang gamemode (nonfunctional)
  • Revs gamemode (nonfunctional)
  • Families gamemode (could you ever call this functional)

Adds:

  • Display order support for crew consoles, based on either job datum or order in the .json file
  • Returning someone from ajail returns them to their original ship

Why It's Good For The Game

SSJob breaks a lot, and just isn't used. This is another piece of global state that's being removed and replaced with something that better fits us.

Changelog

🆑
del: Gang and revs gamemode (the gear remains)
admin: Removes SSjob, occupations are now stored in a global list
/:cl:

@github-actions github-actions bot added DME Edit Admin They do it for free. Code change Watch something violently break. labels Jul 5, 2023
@Anderson-Cooper
Copy link

image

amazing name

@github-actions github-actions bot added the Merge Conflict Use Git Hooks, you're welcome. label Jul 9, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Jul 9, 2023

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

@MarkSuckerberg MarkSuckerberg force-pushed the how-deep-does-the-rabbit-hole-go branch from 7d399bb to a21a946 Compare July 9, 2023 18:41
@github-actions github-actions bot removed the Merge Conflict Use Git Hooks, you're welcome. label Jul 9, 2023
@MarkSuckerberg
Copy link
Member Author

honestly forgot I even did this lmao

@MarkSuckerberg MarkSuckerberg marked this pull request as ready for review July 21, 2023 02:41
@Imaginos16
Copy link
Member

Finish it already or Im putting you on the lizard stretcher

@github-actions github-actions bot added the Merge Conflict Use Git Hooks, you're welcome. label Aug 1, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Aug 1, 2023

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

@MarkSuckerberg MarkSuckerberg requested review from a team as code owners September 13, 2023 18:14
@github-actions github-actions bot added Config Watch us shake head admins violently until this gets merged. Map Change Tile placing is hard. Thank you for your service. Sprites A bikeshed full of soulless bikes. TGUI Sound USSR Anthem 2.66 MB file. Dependencies GitHub Our very own Babylon. labels Sep 13, 2023
@MarkSuckerberg MarkSuckerberg force-pushed the how-deep-does-the-rabbit-hole-go branch from a10f15a to f7cca49 Compare September 13, 2023 18:17
@github-actions github-actions bot added Merge Conflict Use Git Hooks, you're welcome. and removed Merge Conflict Use Git Hooks, you're welcome. Config Watch us shake head admins violently until this gets merged. Map Change Tile placing is hard. Thank you for your service. Sprites A bikeshed full of soulless bikes. Sound USSR Anthem 2.66 MB file. Dependencies GitHub Our very own Babylon. labels Sep 13, 2023
@github-actions
Copy link
Contributor

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

@github-actions github-actions bot removed the Merge Conflict Use Git Hooks, you're welcome. label Sep 14, 2023
@MarkSuckerberg MarkSuckerberg added this pull request to the merge queue Sep 14, 2023
Merged via the queue into shiptest-ss13:master with commit 5dc5ca1 Sep 14, 2023
10 checks passed
@MarkSuckerberg MarkSuckerberg deleted the how-deep-does-the-rabbit-hole-go branch September 14, 2023 16:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Admin They do it for free. Code change Watch something violently break. DME Edit TGUI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants