Remove "--ignore-unsound-tests" from "arc diff"
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F15466013: D21303.id50720.diff
	Thu, Apr 3, 12:49 AM

	F15464393: D21303.id50778.diff
	Wed, Apr 2, 9:07 AM

	F15461798: D21303.id.diff
	Tue, Apr 1, 11:22 AM

	F15459614: D21303.diff
	Mon, Mar 31, 2:36 PM

	F15399380: D21303.id50720.diff
	Mon, Mar 17, 4:18 AM

	F15381113: D21303.diff
	Fri, Mar 14, 5:57 AM

	F15332699: D21303.diff
	Fri, Mar 7, 7:20 PM

	Unknown Object (File)
	Feb 18 2025, 3:47 AM

View All Files

Subscribers

None

Details

Reviewers: None
Maniphest Tasks: T13544: Modernize the "arc diff" workflow
Commits: rARC86471fc0feac: Remove "--ignore-unsound-tests" from "arc diff"

Summary

Ref T13544. This flag only disables a warning and should be a prompt default, not a flag.

Test Plan

Grepped for "ignore-unsound-tests", created this revision.

Diff Detail

Repository

rARC Arcanist

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

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

Harbormaster completed remote builds in B24542: Diff 50720.May 30 2020, 10:50 PM

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

epriestley added a child revision: D21304: Remove "--amend-all", "--no-amend", and "--amend-autofixes" from "arc diff".Jun 3 2020, 7:48 PM

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

Closed by commit rARC86471fc0feac: Remove "--ignore-unsound-tests" from "arc diff" (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rARC86471fc0feac: Remove "--ignore-unsound-tests" from "arc diff".

Revision Contents
Changeset List

Path

Size

src/

workflow/

ArcanistDiffWorkflow.php

31 lines

Diff 50778

View Options

src/workflow/ArcanistDiffWorkflow.php

Remove "--ignore-unsound-tests" from "arc diff"ClosedPublicActions