Page MenuHomePhabricator

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

Authored by epriestley on May 13 2016, 6:30 PM.
Tags
None
Referenced Files
F12841526: D15916.diff
Thu, Mar 28, 9:14 PM
Unknown Object (File)
Wed, Mar 27, 10:10 PM
Unknown Object (File)
Wed, Mar 27, 2:34 PM
Unknown Object (File)
Tue, Mar 26, 2:00 PM
Unknown Object (File)
Sat, Mar 16, 8:26 AM
Unknown Object (File)
Fri, Mar 1, 1:05 AM
Unknown Object (File)
Feb 21 2024, 7:30 AM
Unknown Object (File)
Feb 18 2024, 1:26 AM
Subscribers
None

Details

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.

Screen Shot 2016-05-13 at 11.24.54 AM.png (157×516 px, 25 KB)

Screen Shot 2016-05-13 at 11.24.58 AM.png (108×434 px, 16 KB)

Screen Shot 2016-05-13 at 11.25.06 AM.png (211×405 px, 29 KB)

Diff Detail

Repository
rP Phabricator
Branch
owners9
Lint
Lint Passed
SeverityLocationCodeMessage
Advicesrc/applications/differential/editor/DifferentialTransactionEditor.php:1574XHP16TODO Comment
Unit
Tests Passed
Build Status
Buildable 12192
Build 15394: Run Core Tests
Build 15393: arc lint + arc unit

Event Timeline

epriestley retitled this revision from to Support "Review Changes" and "Block Changes" settings for Owners package "Auto Review".
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.May 13 2016, 7:40 PM
This revision was automatically updated to reflect the committed changes.