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

feat: assign random reviewer command #618

Merged
merged 4 commits into from
Jan 19, 2023
Merged

feat: assign random reviewer command #618

merged 4 commits into from
Jan 19, 2023

Conversation

shay2025
Copy link
Contributor

@shay2025 shay2025 commented Jan 16, 2023

Description

This pull request introduces the $assignRandomReviewer command.
Syntax:
/reviewpad assign-random-reviewer

Related issue

Closes #419

Type of change

New feature (non-breaking change which adds functionality)

How was this tested?

Manual tests and ran the task check cmd.

Checklist

  • I have performed a self-review of my code
  • I have added tests that prove my fix is effective or that my feature works
  • I have run task check -f and have no issues

Code review and merge strategy (ship/show/ask)

  • Ask: this pull request requires a code review before merge

@reviewpad reviewpad bot added medium medium pull request ask Pull request requires a code review before merge run-build Label that controls when build should be executed labels Jan 16, 2023
@reviewpad reviewpad bot requested a review from marcelosousa January 16, 2023 11:26
@reviewpad reviewpad bot added the waiting-review PR waiting for review label Jan 16, 2023
@shay2025
Copy link
Contributor Author

Before merging this pull request, the PR https://github.com/reviewpad/docs/pull/173 needs to be merged first.

Comment on lines 16 to 17
Short: "Assign random reviewer to a pull request",
Long: "Assigns a random user of the GitHub organization as the reviewer.",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Short: "Assign random reviewer to a pull request",
Long: "Assigns a random user of the GitHub organization as the reviewer.",
Short: "Assign a random reviewer to a pull request",
Long: "Assign a random user from the GitHub organization as a pull request reviewer",

@reviewpad reviewpad bot added the requires-author-attention Pull request requires the author to take action label Jan 17, 2023
@reviewpad reviewpad bot removed the requires-author-attention Pull request requires the author to take action label Jan 19, 2023
@ferreiratiago ferreiratiago merged commit a24d5e5 into main Jan 19, 2023
@ferreiratiago ferreiratiago deleted the feat/419 branch January 19, 2023 09:15
@reviewpad
Copy link

reviewpad bot commented Jan 19, 2023

📈 Pull Request Metrics

💻 Coding Time: 55 minutes
🛻 Pickup Time: 1 day
👀 Review Time: 1 day

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ask Pull request requires a code review before merge medium medium pull request run-build Label that controls when build should be executed waiting-review PR waiting for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

assign-random-reviewer command
2 participants