Page MenuHomePhabricator
Create Task
Maniphest T4129

Add limit setting to Futures() call in C# linter
Closed, ResolvedPublic
Actions

Description

Removing the ->limit() call seemed like a great idea right up until the point where I was running it on a code base today, and it started 350 processes on a Windows machine. Needless to say, things did not end well.

Revisions and Commits

rARC Arcanist
ClosedD7607 Set limit for execution in C# linter
AbandonedD7606 Add 'lint.limit' option and use it in linters

Event Timeline

Unknown Object (User) created this task.Nov 19 2013, 6:38 AM
Unknown Object (User) assigned this task to hach-que.
Unknown Object (User) raised the priority of this task from to High.
Unknown Object (User) updated the task description. (Show Details)
Unknown Object (User) added a project: Arcanist.
Unknown Object (User) added subscribers: Unknown Object (User), epriestley.
hach-que edited this Maniphest Task.Nov 19 2013, 7:17 AM
Unknown Object (User) edited this Maniphest Task.Nov 19 2013, 10:04 PM
Unknown Object (User) closed this task as Resolved.Nov 19 2013, 10:04 PM
epriestley added a comment.Nov 19 2013, 10:05 PM

For future reference:

Screen_Shot_2013-11-19_at_1.32.08_PM.png (1×1 px, 223 KB)

Phabricator · Built by Phacility