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

Increase Trait Limit From 10 To 14 #1446

Merged
merged 1 commit into from
Jan 7, 2025

Conversation

angelofallars
Copy link
Contributor

Description

When #720 (PR to raise trait limit from 5 to 10) was merged, we only had 53 traits. Now, we have a total of 109 traits. I was conservative with the trait limit back then because we didn't have as much traits, but now that we have so many we should raise the default trait limit from 10 to 14. I believe 14 should be as high as it gets from now on.

For reference, the quirk limit in /tg/ station is just a maximum of 6 positive quirks. So a typical build that reaches the 14 trait limit would be something like 6 positive traits, 6 negative traits to afford all of those positive traits, 1 language trait and 1 accent trait.

Changelog

🆑 Skubman

  • tweak: The maximum trait limit has increased from 10 traits to 14 traits.

@github-actions github-actions bot added Changes: C# Changes any cs files Changes: YML Changes any yml files labels Jan 6, 2025
Copy link
Contributor

@Remuchi Remuchi left a comment

Choose a reason for hiding this comment

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

I like it

@CerberusWolfie
Copy link
Contributor

This is a great idea with the many traits that are offered. I would like to see this as well.

Copy link
Member

@VMSolidus VMSolidus left a comment

Choose a reason for hiding this comment

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

Yea I also agree. The limit is so tight that players outright stopped taking the smaller traits, which was kind of nuts when we apparently now have 109 traits?!

@VMSolidus VMSolidus merged commit 35afacf into Simple-Station:master Jan 7, 2025
11 of 12 checks passed
SimpleStation14 added a commit that referenced this pull request Jan 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: C# Changes any cs files Changes: YML Changes any yml files
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants