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

already contributed before message #443

Open
aborruso opened this issue Feb 21, 2022 · 5 comments
Open

already contributed before message #443

aborruso opened this issue Feb 21, 2022 · 5 comments
Labels
bug Something isn't working pinned

Comments

@aborruso
Copy link

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?

@aborruso aborruso added the bug Something isn't working label Feb 21, 2022
@ManuelRauber
Copy link

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

@ManuelRauber
Copy link

@tenshiAMD
Also uninstall and reinstall the bot does not help.

@ManuelRauber
Copy link

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
But as you can see, the bot tries to update from an empty contributors list.
This does not make sense at all.

@ManuelRauber
Copy link

Ok, i found it out: It reopened its old PR and tried to update from there.

Solution is:

  1. Close all bot-related PRs
  2. Make sure no branch exists anymore (they all are named like all-contributors/add-{username}) - just delete them
  3. now you can ping the bot again to add someone

@tenshiAMD
Copy link
Member

tenshiAMD commented Oct 18, 2022

@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.

@tenshiAMD tenshiAMD transferred this issue from all-contributors/all-contributors Oct 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working pinned
Projects
None yet
Development

No branches or pull requests

3 participants