ReplyBot
Qwerty-Space opened this issue · comments
I'm fairly new to using GitHub, so I didn't know where else to put this.
I would like to change the ReplyBot to only reply to people in the KEYAUTHORS section as opposed to blacklisting them. How would I go about doing this?
Actually, that's what it does!
line 82 says:
all(auth.lower() != pauthor for auth in KEYAUTHORS):
This will check pauthor against each item in KEYAUTHORS, and if all of them are !=, then it proceeds onto the next line, which is continue.
Personally I've always felt that continue should have been called something else, because what it actually does is skip the rest of this loop and work on the next item. If you're assuming it means to move forward, that's where the misunderstanding is. As always, I'd encourage you to test this out and if you think that the KEYAUTHORS list isn't working properly you can let me know.
If you have any other questions or need code modifications like this, you can PM me on reddit. That'll be better than a github issue.