-
Notifications
You must be signed in to change notification settings - Fork 69
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
Oppiabot should be more proactive towards new contributors #166
Comments
@vojtechjelinek please can you assign me this issue. |
@vojtechjelinek I would like to work on this issue. |
@vojtechjelinek I am a bit confused here. I was thinking of adding a welcome message when a user makes his first PR. If the user doesn't leave any comment on the PR then Oppiabot should ping them to request a review.?? |
I think this is a bit more open-ended issue, basically, for new contributors, only the Oppiabot should make the process of their first PR as easy as possible. Meaning that the contributors should not be tucked on anything and if they are Oppiabot helps them get unstuck or calls someone for help. |
@riyasingh1004 There is no need to leave a welcome message when a user makes their first PR. We don't want to incentivize spammy PRs. It may be fine to leave a message on "first PR merge" though (although it might be better if an actual human does that). @vojtechjelinek please could you lay out the requirements clearly for this issue? It should not be left open-ended as per your last comment. |
Next step: onboarding team to analyze first-time PRs, make a list of common issues, and suggest fixes to Oppiabot. |
Requirements:
|
For PR authors who are new users: when they respond to reviews, and fail to leave a top-level comment, Oppiabot should ping the author and ask them to either say they’re still working on this, or @-mention the person they want a review from.
The text was updated successfully, but these errors were encountered: