Page MenuHomePhabricator

Avoid double [y/N] when updating a not owned revision
ClosedPublic

Authored by Luke081515.2 on Aug 17 2016, 4:55 PM.
Referenced Files
F12183514: D16415.diff
Thu, Sep 14, 4:13 PM
F12180842: D16415.id39474.diff
Tue, Sep 12, 2:58 PM
F12169143: D16415.diff
Thu, Sep 7, 7:16 AM
F12157216: D16415.diff
Tue, Sep 5, 4:09 AM
F12144445: D16415.id39474.diff
Thu, Aug 31, 5:13 AM
F12140603: D16415.id39476.diff
Tue, Aug 29, 2:36 AM
Unknown Object (File)
Aug 12 2023, 4:14 PM
Unknown Object (File)
Aug 12 2023, 4:08 AM
Subscribers

Details

Summary

Since 737f5c0df976fe2b3178aac6ab7feb3d3621d99e arcanist shows [y/N] two times, when arcanist asks you, if you want to proceed, if you want to update a not owned revision. Ths patch fixes
that, so that Arcanist shows [y/N] only once, like at other situations, when Arcanist asks you a question.
Ref T11489

Test Plan

Updated a not owned revision.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Luke081515.2 retitled this revision from to Avoid double [y/N] when updating a not owned revision.
Luke081515.2 updated this object.
Luke081515.2 edited the test plan for this revision. (Show Details)
Luke081515.2 added a project: Arcanist.

I hope this is ok, since this is my first patch actually...

This revision is now accepted and ready to land.Aug 17 2016, 5:36 PM
This revision was automatically updated to reflect the committed changes.