-
Notifications
You must be signed in to change notification settings - Fork 150
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
already contributed before message #443
Comments
Same question here, I've made the mistake to ask the bot in multiple prs to add a contribution for different people, so I can accept them after all at once, but that will lead to merge conflicts. So I closed all PRs from the bot and wanted to recreate them, but nope. "Has contributed before" without being in the .all-contributorsrc file |
@tenshiAMD |
Ok, now I've asked the bot to add another contribution for the same people and it did a super strange PR: https://github.com/BoundfoxStudios/community-project/pull/112/files This PR uses old code to update, because in the current contributors-file there are already people in it: https://github.com/BoundfoxStudios/community-project/blob/5ba79dbbe45e3219f79efcb88ed900e367fcb3cc/.all-contributorsrc |
Ok, i found it out: It reopened its old PR and tried to update from there. Solution is:
|
@ManuelRauber Ok noted. Let's see if I can check this one this week but I cannot promise. PRs are welcome. We need to reproduce some failing tests first. For now, here's a workaround for this one, make sure all PRs are deleted related to that specific contributor and try again. |
Describe the bug
Hi I tried to add a contributor, but I have had already contributed before message.
But he hasn't actually been added yet. He's in a never-approved PR.
To Reproduce
The not approved PR ondata/i-dati-che-vorrei#30
The error message ondata/i-dati-che-vorrei#28 (comment)
Expected behavior
To add him, because he is neither in README nor in .all-contributorsrc
How to add him?
The text was updated successfully, but these errors were encountered: