Page MenuHomePhabricator

Mark the "Reviewer" field for Commits as deprecated
ClosedPublic

Authored by epriestley on Jan 25 2018, 6:28 PM.
Tags
None
Referenced Files
F15532481: D18932.id45425.diff
Wed, Apr 23, 3:59 PM
F15529638: D18932.id45425.diff
Tue, Apr 22, 9:23 PM
F15516582: D18932.diff
Fri, Apr 18, 7:53 PM
F15513776: D18932.diff
Thu, Apr 17, 9:46 PM
F15451402: D18932.id45402.diff
Fri, Mar 28, 9:54 PM
F15429050: D18932.id45402.diff
Mar 24 2025, 12:10 AM
F15421298: D18932.id45425.diff
Mar 21 2025, 10:26 PM
F15410327: D18932.id.diff
Mar 19 2025, 7:04 AM
Subscribers
None

Details

Summary

Depends on D18931. Ref T13048. Ref T13041. This field means "the first accepting reviewer, where order is mostly arbitrary". Modern rules should almost certainly use "Accepting Reviewers" instead.

Getting rid of this completely is a pain, but we can at least reduce confusion by marking it as not-the-new-hotness. Add a "Deprecated" group, move it there, and mark it for exile.

Test Plan

Edited a commit rule, saw it in "Deprecated" group at the bottom of the list:

Screen Shot 2018-01-25 at 10.25.41 AM.png (772×403 px, 88 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable