Remove "--encoding" flag from "arc diff"
ClosedPublic
Actions

Authored by epriestley on May 30 2020, 10:30 PM.

Tags

None

Referenced Files

	F13185433: D21300.diff
	Sat, May 11, 2:54 AM

	F13184957: D21300.id50716.diff
	Sat, May 11, 2:03 AM

	Unknown Object (File)
	Tue, May 7, 5:20 AM

	Unknown Object (File)
	Fri, May 3, 2:42 AM

	Unknown Object (File)
	Thu, May 2, 5:57 PM

	Unknown Object (File)
	Wed, May 1, 6:34 AM

	Unknown Object (File)
	Mon, Apr 29, 2:16 PM

	Unknown Object (File)
	Wed, Apr 24, 10:22 PM

View All 68 Files

Subscribers

None

Details

Reviewers: None
Maniphest Tasks: T13544: Modernize the "arc diff" workflow
Commits: rARCb689a8c78b6c: (stable) Remove "--encoding" flag from "arc diff"
rARC466de2d2e1ec: Remove "--encoding" flag from "arc diff"

Summary

Ref T13544. This flag is generally questionable, likely has no actual uses, and is slated for obsoletion and replacement elsewhere (see T13338).

Test Plan

Grepped for "encoding", found no relevant hits.

Diff Detail

Repository

rARC Arcanist

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.May 30 2020, 10:30 PM

Harbormaster completed remote builds in B24538: Diff 50716.May 30 2020, 10:30 PM

epriestley requested review of this revision.May 30 2020, 10:30 PM

This revision was not accepted when it landed; it landed in state Needs Review.Jun 5 2020, 8:27 PM

Closed by commit rARC466de2d2e1ec: Remove "--encoding" flag from "arc diff" (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rARC466de2d2e1ec: Remove "--encoding" flag from "arc diff".

epriestley added a commit: rARCb689a8c78b6c: (stable) Remove "--encoding" flag from "arc diff".Jun 7 2020, 1:44 PM

Revision Contents
Changeset List

Path

Size

src/

workflow/

ArcanistDiffWorkflow.php

27 lines

Diff 50742

View Options

src/workflow/ArcanistDiffWorkflow.php

Remove "--encoding" flag from "arc diff"ClosedPublicActions