Page MenuHomePhabricator

Apply Owners reviewers using ModularTransactions
ClosedPublic

Authored by epriestley on Mar 20 2017, 4:16 PM.
Tags
None
Referenced Files
F19017771: D17512.diff
Sun, Nov 23, 9:10 AM
F19015278: D17512.diff
Sat, Nov 22, 11:45 PM
F18861511: D17512.id42121.diff
Nov 2 2025, 3:56 PM
F18784624: D17512.id42137.diff
Oct 13 2025, 1:39 PM
F18771159: D17512.diff
Oct 8 2025, 6:05 PM
F18733510: D17512.id.diff
Sep 30 2025, 10:36 PM
F18679069: D17512.id.diff
Sep 26 2025, 12:30 AM
F18657654: D17512.diff
Sep 23 2025, 2:47 AM
Subscribers
None

Details

Summary

Ref T10967. See that task for some discussion. This lets us do double writes on this pathway.

Test Plan

Set an Owners package to auto-review. Created revisions which triggered it: one with no reviewers (autoreview added); one with the package as a blocking reviewer explicitly (no automatic stuff happened, as expected).

Diff Detail

Repository
rP Phabricator
Branch
rev2
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 16036
Build 21263: Run Core Tests
Build 21262: arc lint + arc unit

Event Timeline

This revision is now accepted and ready to land.Mar 20 2017, 4:39 PM
This revision was automatically updated to reflect the committed changes.