Page MenuHomePhabricator
Differential D21482

Update "arc call-conduit" instructions in Conduit API console for required "--"
ClosedPublic
Actions

Authored by epriestley on Oct 19 2020, 6:54 PM.
Tags
None
Referenced Files
F15377927: D21482.diff
Thu, Mar 13, 11:26 AM
F15343641: D21482.diff
Mon, Mar 10, 12:01 AM
F15331321: D21482.diff
Fri, Mar 7, 8:56 AM
F15285897: D21482.diff
Tue, Mar 4, 4:44 PM
Unknown Object (File)
Wed, Feb 26, 7:53 AM
Unknown Object (File)
Feb 9 2025, 4:13 PM
Unknown Object (File)
Jan 28 2025, 5:41 AM
Unknown Object (File)
Jan 27 2025, 2:34 PM
View All Files
Subscribers
None

Details

Reviewers
None
Maniphest Tasks
T13491: Why does Arcanist require "--"?
Commits
rPb2e96df3a3be: Update "arc call-conduit" instructions in Conduit API console for required "--"
Summary

See PHI1912. Ref T13491. "arc" now requires "--" when stdin is not a TTY; provide this argument for users.

Test Plan

Viewed example in console, saw "--". Executed example.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Oct 19 2020, 6:54 PM
Harbormaster completed remote builds in B24945: Diff 51130.Oct 19 2020, 6:55 PM
epriestley requested review of this revision.Oct 19 2020, 6:55 PM
This revision was not accepted when it landed; it landed in state Needs Review.Oct 19 2020, 7:02 PM
Closed by commit rPb2e96df3a3be: Update "arc call-conduit" instructions in Conduit API console for required "--" (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.
epriestley added a commit: rPb2e96df3a3be: Update "arc call-conduit" instructions in Conduit API console for required "--".

Revision Contents
Changeset List

PathSize
src/
applications/
conduit/
controller/
2 lines

Diff 51131

View Options

src/applications/conduit/controller/PhabricatorConduitController.php

Loading...
Phabricator · Built by Phacility