Permalink
Browse files

Supressed PHPMD warnings for too many public mehtods on Options

Removed finally keyword because it was causing issues with phpunit and didn't serve any critical purpose.
Removed PHP 7 from allowed failures on Travis
  • Loading branch information...
gsomoza committed Nov 5, 2015
1 parent 92861a1 commit cdd1ea4dc0883e7be8f6f0a7ce4c8967bfef28e9
Showing with 3 additions and 5 deletions.
  1. +0 −1 .travis.yml
  2. +1 −1 composer.json
  3. +1 −2 src/Migration/Command/Middleware/TransactionMiddleware.php
  4. +1 −1 src/Migration/Options.php
View
@@ -11,7 +11,6 @@ php:
matrix:
allow_failures:
- php: 7.0
- php: hhvm
include:
- php: 5.5
View
@@ -25,7 +25,7 @@
}
},
"require-dev": {
"phpunit/phpunit": "*",
"phpunit/phpunit": "~4.1|^5.0",
"mockery/mockery": "^0.9.4",
"squizlabs/php_codesniffer": "2.*",
"phploc/phploc": "^2.1",
@@ -47,8 +47,7 @@ protected function doExecute(MigrateCommand $command, callable $next)
$migration->finish();
} catch (\Exception $e) {
$migration->abort($e);
} finally {
return $result;
}
return $result;
}
}
@@ -24,7 +24,7 @@
/**
* @{inheritdoc}
*
* @SuppressWarnings(PHPMD.TooManyMethods)
* @SuppressWarnings(PHPMD.TooManyPublicMethods)
*
* @author Gabriel Somoza <gabriel@strategery.io>
*/

0 comments on commit cdd1ea4

Please sign in to comment.