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

german memes #455

Merged
merged 2 commits into from Dec 31, 2023
Merged

german memes #455

merged 2 commits into from Dec 31, 2023

Conversation

ghost
Copy link

@ghost ghost commented Dec 28, 2023

why was is rejected?

@ghost
Copy link
Author

ghost commented Dec 29, 2023

merge when?

@Mikeai
Copy link
Contributor

Mikeai commented Dec 29, 2023

Previously rejected for being there for a few months without comment or mention, these tend to get merged by demand or relevance which I didn't see either at the time. Jokes that are in a different language in probably exactly one use-case seem like a bad idea, but if this is something people want it can be merged.

@ghost
Copy link
Author

ghost commented Dec 31, 2023

rejecting the pull request saying that nobody will use it kinda sets a chicken and egg problem. according to your logic then no chatsounds could be added because they didnt get used before. also there are pull request from other people that also had no comments and got merged so your argument saying that pull request need comments or relevance is kinda weak since only a small portion of meta is actually looking up pull requests on the chatsound github page let alone commenting them. why not merge and remove the chatsounds in a future commit if they are actually rarely used?

@Earu
Copy link
Contributor

Earu commented Dec 31, 2023

Names are not common words used for other chatsounds, the format of the files is ok, its only 4 sounds, merging.

@Earu Earu merged commit fd13746 into Metastruct:master Dec 31, 2023
1 check passed
@ghost
Copy link
Author

ghost commented Dec 31, 2023

thank you

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.

2 participants