Page MenuHomePhabricator
Differential D7377

Provide ConfigurationManager to LintEngine in Arcanist
ClosedPublic
Actions

Authored by epriestley on Oct 21 2013, 10:27 PM.
Tags
None
Referenced Files
F15533461: D7377.id16602.diff
Wed, Apr 23, 7:49 PM
F15529771: D7377.id.diff
Tue, Apr 22, 10:20 PM
F15528848: D7377.id.diff
Tue, Apr 22, 2:25 PM
F15528315: D7377.id16600.diff
Tue, Apr 22, 10:22 AM
F15525825: D7377.diff
Mon, Apr 21, 5:07 PM
F15491969: D7377.id.diff
Sat, Apr 12, 9:26 AM
F15485666: D7377.id.diff
Thu, Apr 10, 1:37 AM
F15341324: D7377.id16600.diff
Mar 9 2025, 7:41 PM
View All Files
Subscribers
aran
csilvers

Details

Reviewers
btrahan
Commits
rARCddbc14ade196: Provide ConfigurationManager to LintEngine in Arcanist
Summary

Unbreaks ArcanistSingleLintEngine / ArcanistScriptAndRegexLinter from recent config churn.

Test Plan

arc lint --engine ArcanistSingleLintEngine --rev HEAD^

Diff Detail

Branch
singlelint
Lint
Lint Passed
Unit
Tests Passed

Event Timeline

csilvers added a comment.Oct 21 2013, 10:33 PM

Looks good to me!, fwiw. Any commit that gets rid of a todo is good by my book (whether or not it actually fixes the thing the todo was about...)

btrahan accepted this revision.Oct 21 2013, 11:55 PM

Just revisited the configuration manager on another diff. <3 it

src/lint/engine/ArcanistLintEngine.php
73

this thing is cool.

epriestley closed this revision.Oct 21 2013, 11:58 PM

Closed by commit rARCddbc14ade196 (authored by @epriestley).

Revision Contents
Changeset List

Diff 16600

View Options

src/lint/engine/ArcanistLintEngine.php

Loading...
View Options

src/lint/engine/ArcanistSingleLintEngine.php

Loading...
View Options

src/lint/linter/ArcanistScriptAndRegexLinter.php

Loading...
View Options

src/workflow/ArcanistLintWorkflow.php

Loading...
Phabricator · Built by Phacility