revi-archive/phorge-arcanist
1
0
Fork 0
mirror of https://we.phorge.it/source/arcanist.git synced 2024-11-22 23:02:41 +01:00
Code Issues Releases Wiki Activity

Fix "arc linters" if no engine is built or configured

Browse source 
Summary: `$engine` will be undefined here. This was just copy/paste derp, combined with me accidentally having an `.arclint` file in a place I didn't expect to have one when I tried to test this.

Test Plan: Ran `arc linters` in a totally bare, non-.arclint-having working copy

Reviewers: btrahan

Reviewed By: btrahan

Subscribers: epriestley

Differential Revision: https://secure.phabricator.com/D9255
This commit is contained in:
epriestley 2014-05-22 10:36:31 -07:00
parent e0ee3e8a07
commit f64dee022e
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/workflow/ArcanistLintersWorkflow.php
View file

@ -43,7 +43,7 @@ EOTEXT
try { try {
$built = $this->newLintEngine()->buildLinters(); $built = $this->newLintEngine()->buildLinters();
} catch (ArcanistNoEngineException $ex) { } catch (ArcanistNoEngineException $ex) {
$built = $engine->buildLinters(); $built = array();
} }
// Note that an engine can emit multiple linters of the same class to run // Note that an engine can emit multiple linters of the same class to run