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
Unknown Object (File)
Sat, Mar 16, 8:50 AM
Unknown Object (File)
Fri, Mar 1, 1:37 AM
Unknown Object (File)
Wed, Feb 21, 8:21 AM
Unknown Object (File)
Feb 14 2024, 10:20 PM
Unknown Object (File)
Feb 3 2024, 11:40 AM
Unknown Object (File)
Feb 3 2024, 11:40 AM
Unknown Object (File)
Feb 3 2024, 11:40 AM
Unknown Object (File)
Jan 29 2024, 6:30 PM
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
Branch
master
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 13350
Build 17127: arc lint + arc unit

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.