Page MenuHomePhabricator

Remove the "you have not specified reviewers" prompt from the `arc` client
ClosedPublic

Authored by epriestley on Jun 17 2016, 2:52 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, May 7, 6:55 AM
Unknown Object (File)
Fri, May 3, 4:33 AM
Unknown Object (File)
Thu, May 2, 1:34 PM
Unknown Object (File)
Mon, Apr 29, 4:35 PM
Unknown Object (File)
Sat, Apr 27, 10:25 AM
Unknown Object (File)
Thu, Apr 25, 12:36 AM
Unknown Object (File)
Thu, Apr 25, 12:36 AM
Unknown Object (File)
Sun, Apr 21, 7:14 PM
Subscribers
None
Tokens
"Like" token, awarded by avivey.

Details

Summary

Ref T4631. Ref T10939. I don't have any good solutions here; this is perhaps the least-bad one.

  • This prompt is misleading/confusing in the presence of Herald/Owners.
  • This prompt is likely of very little value for experienced reviewers.
  • When it works, this prompt may be of some value for new reviewers, but getting it wrong is probably more confusing than getting it right is helpful, and there is a more accurate version of the warning in the web UI that new users are likely to see.
  • In the long run, this code should not live in the client.
Test Plan

Created this revision without specifying reviewers, probably didn't get prompted.

Diff Detail

Repository
rARC Arcanist
Branch
reviewers1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 12703
Build 16153: Run Core Tests
Build 16152: arc lint + arc unit

Event Timeline

epriestley retitled this revision from to Remove the "you have not specified reviewers" prompt from the `arc` client.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
Owners added a reviewer: Restricted Owners Package.Jun 17 2016, 2:52 PM
epriestley edited reviewers, added: chad; removed: Restricted Owners Package.Jun 17 2016, 2:53 PM

See significant discussion in T4631#181162.

chad edited edge metadata.
This revision is now accepted and ready to land.Jun 17 2016, 3:03 PM
This revision was automatically updated to reflect the committed changes.