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
Unknown Object (File)
Thu, Apr 11, 8:09 AM
Unknown Object (File)
Sun, Apr 7, 8:02 AM
Unknown Object (File)
Tue, Apr 2, 8:41 AM
Unknown Object (File)
Mar 10 2024, 2:40 AM
Unknown Object (File)
Feb 26 2024, 9:30 AM
Unknown Object (File)
Dec 27 2023, 12:40 PM
Unknown Object (File)
Dec 25 2023, 2:52 PM
Unknown Object (File)
Dec 22 2023, 6:59 PM
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