Skip to content

Commit

Permalink
Merge pull request phpbb#5485 from marc1706/ticket/15919
Browse files Browse the repository at this point in the history
[ticket/15919] Don't run lint test against node_modules
  • Loading branch information
Derky committed Jan 2, 2019
2 parents 186259c + 14f9917 commit 409bfaf
Showing 1 changed file with 7 additions and 0 deletions.
7 changes: 7 additions & 0 deletions tests/lint_test.php
Original file line number Diff line number Diff line change
Expand Up @@ -67,6 +67,12 @@ protected function check($root)
{
$files = array();
$dh = opendir($root);

if ($dh === false)
{
return $files;
}

while (($filename = readdir($dh)) !== false)
{
if ($filename == '.' || $filename == '..')
Expand All @@ -89,6 +95,7 @@ protected function check($root)
// PHP Fatal error: Cannot declare class Container because the name is already in use in /var/www/projects/phpbb3/tests/../phpBB/vendor/symfony/dependency-injection/Symfony/Component/DependencyInjection/Tests/Fixtures/php/services1-1.php on line 20
// https://gist.github.com/e003913ffd493da63cbc
dirname(__FILE__) . '/../phpBB/vendor',
dirname(__FILE__) . '/../node_modules',
)))
{
$files = array_merge($files, $this->check($path));
Expand Down

0 comments on commit 409bfaf

Please sign in to comment.