Homec4science

Support "Review Changes" and "Block Changes" settings for Owners package "Auto…

Authored by epriestley <git@epriestley.com> on May 13 2016, 20:17.

Description

Support "Review Changes" and "Block Changes" settings for Owners package "Auto Review"

Summary:
Ref T10939. Fixes T8887. This enables and implements the "review" and "blocking review" options for packages.

This is a bit copy-pastey from DifferentialReviewersHeraldAction, which doesn't feel awesome. I think the right fix is Glorious Infrasturcture, though -- I filed T10967 to track that.

Test Plan:

  • Set package autoreveiw to "Review".
  • Updated, got a reveiwer.
  • Set autoreview to "blocking".
  • Updated, got a blocking reviewer.

{F1311720}

{F1311721}

{F1311722}

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T8887, T10939

Differential Revision: https://secure.phabricator.com/D15916

Details

Committed
epriestley <git@epriestley.com>May 14 2016, 02:22
Pushed
aubortJan 31 2017, 17:16
Parents
rPH52ac242eb3b2: Implement "Auto Review" in packages with a "Subscribe" option
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH332d787dc8f4: Support "Review Changes" and "Block Changes" settings for Owners package "Auto… (authored by epriestley <git@epriestley.com>).May 14 2016, 02:22