Byrd-Station/Byrd
Byrd-Station/Byrd
3
1
Fork 32
Code Issues 11 Pull requests 3 Projects 2 Wiki Activity Actions

PDA CartridgeLoader Prediction #727

Merged
LilithArson merged 1 commit from pda-predict-20260730 into master 2026-07-30 18:54:55 +00:00
Conversation 0 Commits 1 Files changed 51 +1039 -1159
LilithArson commented 2026-07-30 17:27:16 +00:00
Owner
Copy link

Short description

Backport of https://github.com/space-wizards/space-station-14/pull/40302

Why we need to add this

Media (Video/Screenshots)

Checklist

Breaking changes

CartridgeLoaderSystem has been moved to shared, with many of the API signatures changing to reflect newer codebase conventions. Preinstalled programs are no longer specified via the preinstalled: datafield, but rather, are installed using normal container fills into preinstalled-program-container. Support for background programs was removed. AfterInteractEvent and DeviceNetworkPacketEvent are relayed through CartridgeRelayedEvent<T>s instead of CartridgeAfterInteractEvent and CartridgeDeviceNetPacketEvent.

<!-- Link definitions, do not edit --> [guidelines]: https://docs.spacestation14.com/en/general-development/codebase-info/pull-request-guidelines.html [ruleset]: https://rules.byrdstation.space/#h.vigrxrnffwji ## Short description <!-- What do you propose to change with your PR? --> Backport of https://github.com/space-wizards/space-station-14/pull/40302 ## Why we need to add this <!-- What is the reason for adding these changes? Please post links to Discussions as well as Bug Reports here. Please describe how this will change the game balance. --> ## Media (Video/Screenshots) <!--If your PR contains in-game changes you must provide screenshots/videos of the changes.--> ## Checklist <!-- Check boxes for faster reviewing of your PR --> - [x] I do not require assistance to complete the PR. - [x] I have verified that the changes work before posting/requesting review of my PR. - [x] I have added screenshots/videos of the changes, or this PR does not change in-game mechanics. - [x] I have read the [Pull Request and Changelog Guidelines][guidelines] and [Byrd Station's Contributor Ruleset][ruleset]. ## Breaking changes <!-- List any breaking changes, including namespaces, public class/method/field changes, prototype renames; and provide instructions for fixing them. This will be posted in #codebase-changes. --> `CartridgeLoaderSystem` has been moved to shared, with many of the API signatures changing to reflect newer codebase conventions. Preinstalled programs are no longer specified via the `preinstalled:` datafield, but rather, are installed using normal container fills into `preinstalled-program-container`. Support for background programs was removed. `AfterInteractEvent` and `DeviceNetworkPacketEvent` are relayed through `CartridgeRelayedEvent<T>`s instead of `CartridgeAfterInteractEvent` and `CartridgeDeviceNetPacketEvent`.
PDA CartridgeLoader Prediction
All checks were successful
Close PRs on master / run (pull_request_target) Has been skipped
Details
Labels: Size / size-label (pull_request_target) Successful in 4s
Details
Labels: PR / labeler (pull_request_target) Successful in 4s
Details
CRLF Check / CRLF Check (pull_request) Successful in 2m11s
Details
RGA schema validator / YAML RGA schema validator (pull_request) Successful in 2m53s
Details
RSI Validator / Validate RSIs (pull_request) Successful in 3m4s
Details
Map file schema validator / YAML map schema validator (pull_request) Successful in 10m17s
Details
YAML Linter / YAML Linter (pull_request) Successful in 13m49s
Details
Test Packaging / Test Packaging (pull_request) Successful in 32m23s
Details
PR Changelogs / changelog (pull_request_target) Successful in 51s
Details
d3935955a7
LilithArson deleted branch pda-predict-20260730 2026-07-30 18:54:55 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
bug
This is in regards to a bug.

  
Changes: Audio

   
Changes: Map

   
Changes: No C#

   
Changes: Sprites

   
Changes: UI

   
documentation
Improvements or additions to documentation

  
DON'T MERGE
This is under review, fixes, or needs significant changes.

  
duplicate
This issue or pull request already exists

  
enhancement
New feature or request

  
good first issue
Good for newcomers

  
help wanted
Extra attention is needed

  
holding to vote
This is currently being voted for by the Byrds.

  
invalid
This doesn't seem right

  
question
Further information is requested

  
size/L

   
size/M

   
size/S

   
size/XL

   
size/XS

   
S: Merge Conflict

   
S: Untriaged

   
wontfix
This will not be worked on

No labels
bug
Changes: Audio
Changes: Map
Changes: No C#
Changes: Sprites
Changes: UI
documentation
DON'T MERGE
duplicate
enhancement
good first issue
help wanted
holding to vote
invalid
question
size/L
size/M
size/S
size/XL
size/XS
S: Merge Conflict
S: Untriaged
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
Byrd-Station/Byrd!727
No description provided.