Paths

Table of Contentst

Differential D19730

Allow `buildFutures` and `resolveFutures` to be overridden
ClosedPublic
Actions

Authored by joshuaspence on Oct 4 2018, 12:02 AM.

Tags

None

Referenced Files

	F15528552: D19730.id48916.diff
	Tue, Apr 22, 11:53 AM

	F15509114: D19730.diff
	Wed, Apr 16, 9:44 AM

	F15492904: D19730.id48925.diff
	Sat, Apr 12, 7:32 PM

	F15467773: D19730.id47147.diff
	Thu, Apr 3, 5:56 PM

	F15465690: D19730.id.diff
	Wed, Apr 2, 10:02 PM

	F15464465: D19730.id48916.diff
	Wed, Apr 2, 9:47 AM

	F15461509: D19730.diff
	Tue, Apr 1, 8:49 AM

	F15454856: D19730.id48925.diff
	Sat, Mar 29, 8:52 PM

Subscribers

Details

Reviewers

Group Reviewers

Blessed Reviewers

Summary

I am writing a proxy linter that can be used to wrap any ArcanistExternalLinter and execute all commands within a Docker container (see [[https://github.com/freelancer/flarc/blob/master/src/lint/linter/ArcanistDockerContainerLinterProxy.php |ArcanistDockerContainerLinterProxy]] from [[https://github.com/freelancer/flarc | flarc]]). In order for ArcanistDockerContainerLinterProxy to behave like the ArcanistExternalLinter that is being proxied, final needs to be removed from some methods.

I figured this was reasonable to submit upstream as a similar change (D19630: Allow `willLintPaths` and `didLintPaths` to be overridden) was previously accepted.

Test Plan

N/A

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

joshuaspence created this revision.Oct 4 2018, 12:02 AM

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptOct 4 2018, 12:02 AM

Harbormaster returned this revision to the author for changes because remote builds failed.Oct 4 2018, 12:02 AM

Harbormaster failed remote builds in B20987: Diff 47147!

joshuaspence mentioned this in D17350: Allow external linters to operate on multiple paths in batches.Oct 4 2018, 12:18 AM

Rebase

joshuaspence edited the summary of this revision. (Show Details)May 14 2019, 12:31 AM

Harbormaster completed remote builds in B22789: Diff 48916.May 14 2019, 12:32 AM

joshuaspence requested review of this revision.May 14 2019, 12:32 AM

Herald added a subscriber: Korvin. · View Herald TranscriptMay 14 2019, 12:32 AM

epriestley accepted this revision.May 14 2019, 2:21 PM

This revision is now accepted and ready to land.May 14 2019, 2:21 PM

Thanks for the reviews :)

Closed by commit rARC6a8e76db329e: Allow `buildFutures` and `resolveFutures` to be overridden (authored by joshuaspence). · Explain WhyMay 14 2019, 11:10 PM

This revision was automatically updated to reflect the committed changes.

joshuaspence added an edge: rARC6a8e76db329e: Allow `buildFutures` and `resolveFutures` to be overridden.

joshuaspence mentioned this in D20523: Add a hook method that is called after `ArcanistLinter::setEngine`.May 16 2019, 1:57 AM

Revision Contents
Changeset List

Path

Size

src/

lint/

linter/

ArcanistExternalLinter.php

4 lines

Diff 48925

src/lint/linter/ArcanistExternalLinter.php

Loading...