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

"Pa" works but "Ma" doesn't #183

Open
qwertystop opened this issue Oct 24, 2021 · 3 comments
Open

"Pa" works but "Ma" doesn't #183

qwertystop opened this issue Oct 24, 2021 · 3 comments

Comments

@qwertystop
Copy link

Testable via YAM -> MAY -> MA.

@angstsmurf
Copy link
Contributor

I actually implemented this, but reverted it along with mama and other things in #145.

The main reason for the removal was that I'm just not great at writing description text, let alone text that fits with the style of the original author, and without good descriptions, new objects are kind of pointless. Also, every new createable object tends to allow the creation several other new ones, which will also need descriptions, and so on. Every object makes the game a tiny bit slower, and if you keep adding new ones, the slowdown will eventually become noticeable.

Still, if you have a really good description of a ma and @emshortif approves, I will consider it.

@qwertystop
Copy link
Author

Fair enough. I'd think the same would be true for "pa", and it's mostly the parallel that makes this particular example feel odd (especially since, as in your previous implementation, it doesn't take much of a tweak to the "pa" description to have a "ma" one). But that's reasonable enough regardless, it's not otherwise a particularly interesting thing.

@angstsmurf
Copy link
Contributor

I tend to agree, even if I wish the description was better. And unless I'm missing something, it will still not be possible to create a mama, so we don't need to re-add her.

Does @emshortif have an opinion on this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants