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

Add patch for long pronouns. #54

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

PlaceReporter99
Copy link

@PlaceReporter99 PlaceReporter99 commented Sep 26, 2023

Fixes #53 , even though it's already closed.

This cuts pronouns longer than 13 characters short with an ellipsis.

@Glorfindel83
Copy link
Owner

Thanks for your contribution - it's been a while since I was really active on Stack Exchange, nowadays, after the generative AI issues, and because of real-life matters (nothing too problematic, fortunately), I participate only when I'm bored or have absolutely nothing else to do.

I made a comment on that issue because your pronoun information reminded me of the attack helicopter - I hope I'm correct in assuming you put it there as a joke and are not demanding users actually refer you by those. Now I know more users are putting jokes there, but at least this one is on-topic and not easily confused with a forbidden meme. And there's even one user who uses it for software verification. So I'm not sure if I should accept this pull request (I didn't close the original issue, its author did that.)

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

Successfully merging this pull request may close these issues.

Pronoun Assistent: Limit pronoun display in chat to a certain length
2 participants