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

Doesn't recognize hyphens in a user's name (at the end of their name)? #310

Closed
jpivarski opened this issue May 8, 2020 · 7 comments
Closed
Labels

Comments

@jpivarski
Copy link

See scikit-hep/awkward#238

I attempted to add @nsmith- (name ends in a hyphen) and it didn't work the first time (don't know why), but a trivial change in the list of roles (removed Oxford comma) caused the bot to add @nsmith without a hyphen.

I'm now editing the pull request by hand.

@baikho
Copy link
Contributor

baikho commented May 22, 2020

Hi @jpivarski,
I believe this is a duplicate of #292 and PR #307 . Do we need to tag a new release for these changes to take effect @Berkmann18 ?

@Berkmann18
Copy link
Member

@baikho Normally no, as it's done automatically.
I noticed your PR didn't get a response from the release bot which makes me wonder what went wrong.

@rkdarst
Copy link

rkdarst commented Oct 2, 2020

I just got hit with this: I used the bot to add a contributor with github login firstname-lastname, and the bot added firstname (a different person) as a contributor! rkdarst/all-contributors-test#2

@reobin
Copy link

reobin commented Oct 2, 2020

Seems like this is still an issue, even when using the @ before the user name

Screen Shot 2020-10-02 at 08 58 20

Link

@Berkmann18
Copy link
Member

That seems to be due to the fact that the CI/CD pipeline didn't work on the @baikho's PR and I don't have admin rights there so I can't see exactly what went wrong.

@Berkmann18
Copy link
Member

@jpivarski The bot was rewritten and we've got a better logging system now, could you try again and we'll see if it's still an issue?

@tenshiAMD
Copy link
Member

@jpivarski The bot was rewritten and we've got a better logging system now, could you try again and we'll see if it's still an issue?

@jpivarski Have you check this using latest version? Please check and let us know if this still occurs. Closing this for now. Thanks! 🎉

@tenshiAMD tenshiAMD closed this as not planned Won't fix, can't repro, duplicate, stale Sep 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

6 participants