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

Move getPEP8WithTextOptions method into ComprehensiveLintEngine class.

Browse source 
Summary: Since this method is only used within this class, it makes sense to move it here.

Test Plan: N/A

Reviewers: epriestley, #blessed_reviewers

Reviewed By: epriestley, #blessed_reviewers

Subscribers: epriestley, Korvin

Differential Revision: https://secure.phabricator.com/D9114
This commit is contained in:
Joshua Spence 2014-05-16 19:18:35 -07:00 committed by epriestley
parent 75cb3cf358
commit 38eda3e86b
2 changed files with 9 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
src/lint/engine/ArcanistLintEngine.php
View file

@ -540,15 +540,6 @@ abstract class ArcanistLintEngine {
return 1; return 1;
} }
protected function getPEP8WithTextOptions() {
// E101 is subset of TXT2 (Tab Literal).
// E501 is same as TXT3 (Line Too Long).
// W291 is same as TXT6 (Trailing Whitespace).
// W292 is same as TXT4 (File Does Not End in Newline).
// W293 is same as TXT6 (Trailing Whitespace).
return '--ignore=E101,E501,W291,W292,W293';
}
/** /**
* Get a named linter resource shared by another linter. * Get a named linter resource shared by another linter.

9
src/lint/engine/ComprehensiveLintEngine.php
View file

@ -48,4 +48,13 @@ final class ComprehensiveLintEngine extends ArcanistLintEngine {
return $linters; return $linters;
} }
protected function getPEP8WithTextOptions() {
// E101 is subset of TXT2 (Tab Literal).
// E501 is same as TXT3 (Line Too Long).
// W291 is same as TXT6 (Trailing Whitespace).
// W292 is same as TXT4 (File Does Not End in Newline).
// W293 is same as TXT6 (Trailing Whitespace).
return array('--ignore=E101,E501,W291,W292,W293');
}
} }