Page MenuHomePhabricator

Don't trigger "Auto Review" if the author is already an owner; document "Auto Review"
ClosedPublic

Authored by epriestley on May 13 2016, 6:49 PM.

Details

Summary

Ref T10939. If you already own a package, don't trigger the subscribe/review rules.

Document how these rules work.

Test Plan
  • Read documentation.
  • Removed reviewers, updated a revision, got autoreviewed.
  • Joined package.
  • Removed reveiwers, updated a revision, no more autoreview.

Diff Detail

Repository
rP Phabricator
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

epriestley updated this revision to Diff 38322.May 13 2016, 6:49 PM
epriestley retitled this revision from to Don't trigger "Auto Review" if the author is already an owner; document "Auto Review".
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad accepted this revision.May 13 2016, 7:51 PM
chad edited edge metadata.
This revision is now accepted and ready to land.May 13 2016, 7:51 PM
This revision was automatically updated to reflect the committed changes.