Set limit for execution in C# linter
ClosedPublic
Actions

Authored by hach-que on Nov 19 2013, 9:33 PM.

Tags

None

Referenced Files

	F15472066: D7607.id17166.diff
	Sat, Apr 5, 10:48 AM

	F15471775: D7607.diff
	Sat, Apr 5, 8:03 AM

	F15397816: D7607.id17166.diff
	Sun, Mar 16, 10:26 PM

	F15396948: D7607.id17167.diff
	Sun, Mar 16, 5:09 PM

	F15391647: D7607.id.diff
	Sat, Mar 15, 10:50 AM

	F15389867: D7607.id17167.diff
	Sat, Mar 15, 5:43 AM

	F15389593: D7607.diff
	Sat, Mar 15, 5:29 AM

	F15389267: D7607.id17166.diff
	Sat, Mar 15, 5:03 AM

Subscribers

Details

Reviewers

epriestley

Group Reviewers

Blessed Reviewers

Maniphest Tasks

T4129: Add limit setting to Futures() call in C# linter

Commits

rARCfb8a0d32ae47: Set limit for execution in C# linter

Summary

This sets the limit for future execution in the C# linter to 8. See D7606 for more information.

Test Plan

Ran

arc lint --everything --trace --never-apply-patches --output json

and saw it only run 8 commands at once.

Diff Detail

Branch

linter-csharp-limit

Lint

Lint Passed

Unit

Test Failures

Event Timeline

Works for me.

Closed by commit rARCfb8a0d32ae47 (authored by @hach-que, committed by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

lint/

linter/

ArcanistCSharpLinter.php

2 lines

Diff 17166

View Options

src/lint/linter/ArcanistCSharpLinter.php

Set limit for execution in C# linterClosedPublicActions