Page MenuHomePhabricator

Remove "--use-commit-message/-C" from "arc diff"
ClosedPublic

Authored by epriestley on May 30 2020, 9:34 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Mar 24, 1:55 AM
Unknown Object (File)
Fri, Mar 15, 12:55 PM
Unknown Object (File)
Feb 18 2024, 3:38 AM
Unknown Object (File)
Feb 14 2024, 5:15 AM
Unknown Object (File)
Feb 8 2024, 10:11 AM
Unknown Object (File)
Jan 28 2024, 7:40 AM
Unknown Object (File)
Jan 11 2024, 11:58 AM
Unknown Object (File)
Jan 7 2024, 9:00 PM
Subscribers
None

Details

Summary

Ref T13544. This flag was introduced in D1385 (2012) as part of a workflow which no longer exists. I can't recall anyone ever reporting an issue which involves its use and believe it is likely unused. It's not obvious to me why someone would use it in modern "arc".

(The same goal can be accomplished with "--message-file ...", although this requires more steps.)

Test Plan

Grepped for "use-commit-message" and "getCommitMessageFromCommit", ran "arc diff".

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable