Page MenuHomePhabricator

Add a configuration option for linter future limit
Needs ReviewPublic

Authored by stefren on Aug 25 2017, 1:23 AM.
Tags
None
Referenced Files
F18958710: D18470.id.diff
Thu, Nov 13, 5:19 AM
F18809281: D18470.id44374.diff
Oct 19 2025, 12:38 PM
F18591050: D18470.diff
Sep 12 2025, 8:04 AM
F18584781: D18470.diff
Sep 11 2025, 1:40 PM
F18583209: D18470.id.diff
Sep 11 2025, 9:49 AM
F18583195: D18470.id44374.diff
Sep 11 2025, 9:48 AM
F18583035: D18470.diff
Sep 11 2025, 9:19 AM
F17948964: D18470.id.diff
Jul 31 2025, 10:28 PM
Subscribers

Details

Reviewers
None
Group Reviewers
Blessed Reviewers
Summary

Currently linters are being run using futures, but with a limit of 8 at a time.
Testing indicates that using more futures could improve the runtime of the linter.
Adds a default of 32 futures for the linter, and allows for a config option which
overrides the default.

Test Plan

Ran arc lint --trace and watched PyLint run in ~5 seconds. Set
'parallelism' in .arclint to 8 and watched PyLint run in ~16 seconds.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 18161
Build 24411: Run Core Tests
Build 24410: arc lint + arc unit