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

Oppiabot should be more proactive towards new contributors #166

Open
vojtechjelinek opened this issue Dec 10, 2020 · 7 comments
Open

Oppiabot should be more proactive towards new contributors #166

vojtechjelinek opened this issue Dec 10, 2020 · 7 comments
Labels

Comments

@vojtechjelinek
Copy link
Contributor

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.

@nlok5923
Copy link
Contributor

@vojtechjelinek please can you assign me this issue.

@riyasingh1004
Copy link
Contributor

@vojtechjelinek I would like to work on this issue.

@riyasingh1004
Copy link
Contributor

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

@vojtechjelinek
Copy link
Contributor Author

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.

@seanlip
Copy link
Member

seanlip commented Apr 18, 2021

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

@seanlip
Copy link
Member

seanlip commented Apr 20, 2021

Next step: onboarding team to analyze first-time PRs, make a list of common issues, and suggest fixes to Oppiabot.

@DubeySandeep
Copy link
Member

DubeySandeep commented Apr 28, 2021

Requirements:

  1. Ask one of the reviewers to approve the run build for the first-time contributors. [Maybe ask the same person who adds changelog label to the PRs]
  2. We need to find a way to help to assign reviewers to the PR if the author (new contributor) misses leaving top-level comment or a PTAL comment.
    i. Maybe the bot can send new-contributor-specific comments in the PR that we recommend new-contributors to follow on their PR have better contribution experience. [Discussing this with other teams, will update once we'll have a solution]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Todo
Development

No branches or pull requests

5 participants