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

Make Sweet Tooth a starting trait #33477

Merged
merged 2 commits into from
Aug 24, 2019
Merged

Make Sweet Tooth a starting trait #33477

merged 2 commits into from
Aug 24, 2019

Conversation

ebolite
Copy link
Contributor

@ebolite ebolite commented Aug 22, 2019

Summary

SUMMARY: Content "Make 'Sweet Tooth' a starting trait"

Purpose of change

Sweet Tooth seems trait-like and like a viable candidate for a starting trait. Its counterpart, Junkfood Intolerance, is already a starting trait, and while it is a realistic trait for a normal person to have, Sweet Tooth can currently only be obtained through mutations.

Describe the solution

Sweet Tooth has been changed to be a starting trait, and a new NPC trait group, "trait_group_PROJUNK", has been added to encompass Sweet Tooth and Junkfood Intolerance. It was given a probability of 5 due to the group containing an allergy. Junkfood Intolerance has been removed from the NPC starting traits.

Describe alternatives you've considered

If it is decided that Sweet Tooth stay mutation-only, it should be made clear that it is an unnatural attachment to junk food rather than a preference, like Sugar-Loving. If Sweet Tooth's morale bonus is too high, it could warrant the trait's cost being bumped up to 2, or having the morale bonus lowered.

Additional context

This is my first PR on GitHub, so please tell me if I've done anything wrong.

Removed ANTIJUNK from possible starting traits, created a new trait group with both PROJUNK and ANTIJUNK and added that group to possible starting traits. Possibility 5 because of allergy.
@alkemann alkemann added Mutations / Traits / Professions/ Hobbies Mutations / Traits / Professions/ Hobbies NPC / Factions NPCs, AI, Speech, Factions, Ownership labels Aug 22, 2019
@ZhilkinSerg ZhilkinSerg merged commit 5c7083b into CleverRaven:master Aug 24, 2019
misterprimus pushed a commit to misterprimus/Cataclysm-DDA that referenced this pull request Sep 21, 2019
* Changed Sweet Tooth to be a starting trait

* Added Sweet Tooth starting trait to NPCs

Removed ANTIJUNK from possible starting traits, created a new trait group with both PROJUNK and ANTIJUNK and added that group to possible starting traits. Possibility 5 because of allergy.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Mutations / Traits / Professions/ Hobbies Mutations / Traits / Professions/ Hobbies NPC / Factions NPCs, AI, Speech, Factions, Ownership
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants