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

17 year olds can no longer exist upon Nanotrasen Property, or in the entire universe for that matter #9996

Merged
merged 1 commit into from
Oct 17, 2020

Conversation

swissloaf
Copy link
Contributor

@swissloaf swissloaf commented Oct 15, 2020

Intent of your Pull Request

Raises minimum age of a character to be 18, an intergalactically agreed age of an adult. Agreed on by both the Syndicate and Nanotrasen, so now you know even the Syndicate doesn't support child predators.

Also unironically why would you hire someone under 18, especially for a Head role

Changelog

🆑
tweak: Changes AGE_MIN from 17 to 18
/:cl:

@Yogbot-13 Yogbot-13 added the Tweak This PR contains changes to the game. label Oct 15, 2020
@TheLinuxOS
Copy link

I mean I see no problem with this, makes sense

@jcatjmeow
Copy link
Contributor

jcatjmeow commented Oct 16, 2020

how does this affect those who have characters at 17 when this is merged. I like it but we should make sure it don't break shit

@Tipy1802
Copy link

To be fair an 18 years old captain is as realistic as a 17 year old on.
I would suggest bumping it to 25

@Tipy1802
Copy link

To be fair for a 25 year old captain is also a bit too little but you can’t really push it further without getting an angry mob with pitchforks

@Amelia0010
Copy link
Contributor

how does this affect those who have characters at 17 when this is merged. I like it but we should make sure it don't break shit

age = sanitize_integer(age, AGE_MIN, AGE_MAX, initial(age))
should be fine

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Tweak This PR contains changes to the game.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants