Paths

Table of Contentst

Remove "async lint" from `arc lint`
ClosedPublic
Actions

Authored by epriestley on Sep 25 2017, 6:08 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, Feb 15, 10:22 AM

	Unknown Object (File)
	Wed, Feb 12, 10:13 PM

	Unknown Object (File)
	Tue, Feb 11, 8:47 PM

	Unknown Object (File)
	Sat, Feb 8, 12:01 AM

	Unknown Object (File)
	Thu, Feb 6, 6:08 PM

	Unknown Object (File)
	Thu, Jan 30, 11:18 PM

	Unknown Object (File)
	Sun, Jan 26, 11:42 PM

	Unknown Object (File)
	Wed, Jan 22, 5:33 PM

View All Files

Subscribers

None

Details

Reviewers

amckinley

Maniphest Tasks

T12996: Parallelize substages of `arc diff`

Commits

rARC3453ef835800: Remove "async lint" from `arc lint`

Summary

Ref T12996. This was a very old FB-specific feature that caused some kind of server-side lint thing to run. You can/should do this now with Harbormaster instead. Nothing else ever used it and it has no role in the upstream.

Test Plan

Grepped for asynclint, created this revision.

Diff Detail

Repository

rARC Arcanist

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Sep 25 2017, 6:08 PM

Harbormaster completed remote builds in B18534: Diff 44756.Sep 25 2017, 6:09 PM

amckinley accepted this revision.Sep 25 2017, 6:31 PM

This revision is now accepted and ready to land.Sep 25 2017, 6:31 PM

Closed by commit rARC3453ef835800: Remove "async lint" from `arc lint` (authored by epriestley). · Explain WhySep 27 2017, 3:18 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

lint/

engine/

ArcanistLintEngine.php

10 lines

workflow/

ArcanistLintWorkflow.php

7 lines

Diff 44774

View Options

src/lint/engine/ArcanistLintEngine.php