Page MenuHomePhabricator

Remove "--apply-patches" and "--never-apply-patches" from "arc diff"
Needs ReviewPublic

Authored by epriestley on May 30 2020, 10:47 PM.
Tags
None
Referenced Files
F18827031: D21302.id.diff
Fri, Oct 24, 7:14 AM
F18825676: D21302.diff
Thu, Oct 23, 11:28 PM
F18789590: D21302.diff
Wed, Oct 15, 10:15 AM
F18734313: D21302.id.diff
Tue, Sep 30, 10:51 PM
F18621047: D21302.id.diff
Sep 15 2025, 6:56 AM
F18618023: D21302.diff
Sep 14 2025, 10:58 PM
F18337641: D21302.diff
Aug 25 2025, 6:26 PM
F18187751: D21302.id50719.diff
Aug 16 2025, 8:11 PM
Subscribers
None
This revision needs review, but there are no reviewers specified.

Details

Reviewers
None
Maniphest Tasks
T13544: Modernize the "arc diff" workflow
Summary

Ref T13544. These are passthrough flags that control the behavior of the "arc lint" subprocess.

I suspect all use cases of "--apply-patches" are better served by a prompt default via "arc prompts".

I suspect all use cases of "--never-apply-patches" are better served by either using prompt defaults or by using "--nolint", which accomplishes the same goal ("ignore lint") much faster. If you want to look at lint and make a nuanced decision about what to act on, you can use "arc lint".

Test Plan

Searched for "apply-patches" and "never-apply-patches", created this revision.

Diff Detail

Repository
rARC Arcanist
Branch
xdiff7
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 24541
Build 33823: Run Core Tests
Build 33822: arc lint + arc unit