Skip to content

Added a network configurator to the Warden's locker. #39254

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jul 28, 2025
Merged

Added a network configurator to the Warden's locker. #39254

merged 1 commit into from
Jul 28, 2025

Conversation

@SuperGDPWYL
Copy link
Contributor

@SuperGDPWYL SuperGDPWYL commented Jul 28, 2025 •
edited
Loading

About the PR

Added a network configurator to the Warden's locker.

Why / Balance

It's practically necessary to use the electropacks the Warden starts with and as far as I'm aware (with 75 hours in the role), only Exo actually gives them one at the start of the round. This means the Warden has to go out of their way to get one from Tools to be able to use them at all. Given the Warden is heavily incentivised to remain in Security outside of emergencies, this is not ideal.

Technical details

Added NetworkConfigurator to FillLockerWarden. 1 line YAML change.

Media

image

Requirements

Changelog

🆑

  • add: The Warden now starts with a network configurator in their locker.

@PJBot PJBot added S: Needs Review Status: Requires additional reviews before being fully accepted. Not to be replaced by S: Approved. S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. Changes: No C# Changes: Requires no C# knowledge to review or fix this item. labels Jul 28, 2025
@github-actions github-actions bot added the size/XS Denotes a PR that changes 0-9 lines. label Jul 28, 2025
@lzk228
Copy link
Contributor

lzk228 commented Jul 28, 2025

add the locker part to the cl

@SuperGDPWYL
Copy link
Contributor Author

add the locker part to the cl

? not sure what isn't mentioned in the CL

slarticodefast
slarticodefast approved these changes Jul 28, 2025
View reviewed changes
Copy link
Member

@slarticodefast slarticodefast left a comment

Choose a reason for hiding this comment

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

Makes sense to me!

@PJBot PJBot added S: Approved Status: Reviewed and approved by at least one maintainer; a PR may require another approval. and removed S: Needs Review Status: Requires additional reviews before being fully accepted. Not to be replaced by S: Approved. labels Jul 28, 2025
@slarticodefast slarticodefast added T: Balance Change Type: Balance changes through direct value changes, or changes to mechanics that affect it A: Security Area: Security department, including Detectives, HoS and removed S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. labels Jul 28, 2025
southbridge-fur
southbridge-fur approved these changes Jul 28, 2025
View reviewed changes
Copy link
Contributor

@southbridge-fur southbridge-fur left a comment

Choose a reason for hiding this comment

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

I'm curious as to which locker has the most stuff in it because I feel like the warden's locker is definitely up there.

@southbridge-fur southbridge-fur merged commit b0825c1 into space-wizards:master Jul 28, 2025
12 checks passed
@SuperGDPWYL SuperGDPWYL deleted the warden-network-config branch July 29, 2025 16:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@southbridge-fur southbridge-fur southbridge-fur approved these changes

@slarticodefast slarticodefast slarticodefast approved these changes

Assignees

No one assigned

Labels

A: Security Area: Security department, including Detectives, HoS Changes: No C# Changes: Requires no C# knowledge to review or fix this item. S: Approved Status: Reviewed and approved by at least one maintainer; a PR may require another approval. size/XS Denotes a PR that changes 0-9 lines. T: Balance Change Type: Balance changes through direct value changes, or changes to mechanics that affect it

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@SuperGDPWYL @lzk228 @southbridge-fur @slarticodefast @PJBot