Skip to content

Commit

Permalink
Merge pull request #374 from andres-montanez/revert-357-master
Browse files Browse the repository at this point in the history
Revert "[FEATURE] allow exclude definition global"
  • Loading branch information
andres-montanez committed Apr 14, 2017
2 parents c649814 + 591ac04 commit d95e498
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 39 deletions.
2 changes: 1 addition & 1 deletion src/Task/BuiltIn/Deploy/RsyncTask.php
Expand Up @@ -54,7 +54,7 @@ public function execute()

protected function getExcludes()
{
$excludes = $this->runtime->getMergedOption('exclude', []);
$excludes = $this->runtime->getEnvOption('exclude', []);
$excludes = array_merge(['.git'], array_filter($excludes));

foreach ($excludes as &$exclude) {
Expand Down
2 changes: 1 addition & 1 deletion src/Task/BuiltIn/Deploy/Tar/PrepareTask.php
Expand Up @@ -51,7 +51,7 @@ public function execute()

protected function getExcludes()
{
$excludes = $this->runtime->getMergedOption('exclude', []);
$excludes = $this->runtime->getEnvOption('exclude', []);
$excludes = array_merge(['.git'], array_filter($excludes));

foreach ($excludes as &$exclude) {
Expand Down
22 changes: 2 additions & 20 deletions tests/Command/BuiltIn/DeployCommandMiscTasksTest.php
Expand Up @@ -50,27 +50,9 @@ public function testSymfonyEnvironmentConfiguration()
$this->assertEquals(0, $tester->getStatusCode());
}

public function composerFlagsDataProvider() {
return [
'default' => [
'yml' => __DIR__ . '/../../Resources/composer.yml'
],
'global exclude' => [
'yml' => __DIR__ . '/../../Resources/global-exclude.yml'
],
];
}

/**
* testComposerFlags
*
* @dataProvider composerFlagsDataProvider
* @param string $yml
* @return void
*/
public function testComposerFlags($yml)
public function testComposerFlags()
{
$application = new MageApplicationMockup($yml);
$application = new MageApplicationMockup(__DIR__ . '/../../Resources/composer.yml');

/** @var AbstractCommand $command */
$command = $application->find('deploy');
Expand Down
17 changes: 0 additions & 17 deletions tests/Resources/global-exclude.yml

This file was deleted.

0 comments on commit d95e498

Please sign in to comment.