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

Remove the age requirement on command roles #1611

Merged
4 commits merged into from
Sep 7, 2024

Conversation

sleepyyapril
Copy link
Contributor

@sleepyyapril sleepyyapril commented Aug 12, 2024

About the PR

Remove the age requirement on command roles.

Why / Balance

Due to the server rule of needing their character to be at minimum 17 years old (mentally, I assume), this restriction feels incredibly restrictive to people playing borgs with "manufacture date" as the year they were born. In general, this restriction is unneeded.

Technical details

YML change.

Media

None

Requirements

  • I have read and I am following the Pull Request Guidelines. I understand that not doing so may get my pr closed at maintainer’s discretion
  • I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

🆑

  • tweak: Removed command age requirement.

@github-actions github-actions bot added the Changes: YML Changes any yml files label Aug 12, 2024
Copy link
Contributor

@Colin-Tel Colin-Tel left a comment

Choose a reason for hiding this comment

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

Needs Project Manager approval first.

@Colin-Tel Colin-Tel added the S: Needs Discussion Must be discussed label Aug 12, 2024
@BlitzTheSquishy
Copy link
Contributor

Feels like a non-issue, wouldnt want someone JUST out of highschool running my station, and theres no real reason to up it either

@Alyx-Wander
Copy link

As a project manager, my own opinion is that if this is to be changed at all, it should be towards the removal of the limit rather than lowering it to an arbitrary amount. It doesn't really add anything mechanically or in terms of roleplay, but does have the ability to exclude players who'm may want to play with non-human life-spans. As an example, synethics who set their age as their date of manufacture, some of the awakened animal characters we have (Soup is an example, although they don't play Command they do play a character with a natural cat's lifespan), and doubtless other examples.

Instead of lowering the requirement to an arbitrary number, we think it'd just be better to outright remove it for the net-zero gain the feature gives us, and the real exclusionary downside it provides as well. Even excluding a handful of players seems like too steep a price for no-benefit.

@sleepyyapril sleepyyapril changed the title Lower age requirement of command roles to 18. Remove the age requirement on command roles Aug 13, 2024
Copy link
Member

@deltanedas deltanedas left a comment

Choose a reason for hiding this comment

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

these all need deltav comments and commenting out instead of removal

Resources/Prototypes/Roles/Jobs/Cargo/quartermaster.yml Outdated Show resolved Hide resolved
@sleepyyapril sleepyyapril reopened this Aug 14, 2024
Copy link
Contributor

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

@github-actions github-actions bot added the S: Merge Conflict Fix your PR! label Aug 18, 2024
@TadJohnson00
Copy link
Contributor

Is this still being worked on @flyingkarii ?

@Samsterious
Copy link
Contributor

Samsterious commented Sep 5, 2024

This is one of those debates that is a result of a feature nobody asked for or needed in the first place. Creating an IC age requirement to play a certain role is a strange restriction in a server that includes players 17 and up, and the "highschoolers in command" argument is an easy knee-jerk response but does not actually mean anything in the context of roleplay standards.

The reality is that we have numerous extant characters on this server that are directly harmed by this mechanic: synthetic, genetically-engineered, anomalous, and littlespace characters are all at least partially excluded from Command. These types of characters are most often written by our best roleplayers, not our worst.

Nobody in real life is actually benefiting from this mechanic; its benefits are seen purely within the fiction of the world. It betrays the ethos of Delta-V by excluding a queer identity that is expressed by several of our players, including some of our administration. This PR, or one like it, should be merged with urgency.

ghost
ghost previously approved these changes Sep 6, 2024
@ghost ghost dismissed their stale review via c09e3b8 September 6, 2024 21:37
@github-actions github-actions bot removed the S: Merge Conflict Fix your PR! label Sep 6, 2024
@ghost
Copy link

ghost commented Sep 6, 2024

@deltanedas your requested changes are preventing this from being merged and I have no clue why

@ghost ghost merged commit 59cf511 into DeltaV-Station:master Sep 7, 2024
11 checks passed
@sleepyyapril sleepyyapril deleted the revertagerequirements branch September 7, 2024 08:07
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: YML Changes any yml files S: Needs Discussion Must be discussed S: Needs Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants