Skip to content
GitLab Next
  • Menu
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • GitLab FOSS GitLab FOSS
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 0
    • Issues 0
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
    • Requirements
  • Merge requests 1
    • Merge requests 1
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages & Registries
    • Packages & Registries
    • Package Registry
    • Container Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Metrics
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • Code review
    • Insights
    • Issue
    • Repository
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Commits
  • Issue Boards
Collapse sidebar
  • GitLab.orgGitLab.org
  • GitLab FOSSGitLab FOSS
  • Merge requests
  • !24938

Reviewer roulette via Danger

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Nick Thomas requested to merge 56087-danger-roulette into master Feb 05, 2019
  • Overview 88
  • Commits 3
  • Pipelines 30
  • Changes 9

What does this MR do?

(Tries to) add reviewer roulette via Danger. The idea is that danger can scan the MR and determine what reviews it needs. Once decided, it can randomly pick a reviewer and maintainer for each category, perhaps taking OOO status into account along with any other information. It could even take CODEOWNERS information into account.

The final choice of which reviewers to use should remain with the author of the merge request - these are just suggestions to ease selection when the author doesn't have a strong opinion.

I'm new to danger ✨ - let's see what I can do!

What are the relevant issue numbers?

Closes #56087 (closed)

Does this MR meet the acceptance criteria?

  • [-] Changelog entry added, if necessary
  • Documentation created/updated via this MR
  • Tests added for this feature/bug
  • Conforms to the code review guidelines
  • Conforms to the style guides
  • Security reports checked/validated by reviewer
Edited Feb 13, 2019 by Nick Thomas
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: 56087-danger-roulette