Skip to content

Copy extra.symfony.require when unpacking#443

Merged
fabpot merged 1 commit intosymfony:masterfrom
nicolas-grekas:unpack-wildcard
Dec 11, 2018
Merged

Copy extra.symfony.require when unpacking#443
fabpot merged 1 commit intosymfony:masterfrom
nicolas-grekas:unpack-wildcard

Conversation

@nicolas-grekas
Copy link
Copy Markdown
Member

eg when unpacking the test pack:

--- a/composer.json
+++ b/composer.json
@@ -5,11 +5,14 @@
         "php": "^7.1.3",
         "ext-ctype": "*",
         "ext-iconv": "*",
+        "symfony/browser-kit": "4.2.*",
         "symfony/console": "4.2.*",
+        "symfony/css-selector": "4.2.*",
         "symfony/dotenv": "4.2.*",
         "symfony/flex": "^1.1",
         "symfony/framework-bundle": "4.2.*",
-        "symfony/test-pack": "^1.0",
+        "symfony/panther": "*",
+        "symfony/phpunit-bridge": "4.2.*",
         "symfony/yaml": "4.2.*"
     },

Comment thread src/Unpacker.php
@fabpot
Copy link
Copy Markdown
Member

fabpot commented Dec 11, 2018

Thank you @nicolas-grekas.

@fabpot fabpot merged commit 7f685f6 into symfony:master Dec 11, 2018
fabpot added a commit that referenced this pull request Dec 11, 2018
This PR was merged into the 1.1-dev branch.

Discussion
----------

Copy extra.symfony.require when unpacking

eg when unpacking the test pack:

```diff
--- a/composer.json
+++ b/composer.json
@@ -5,11 +5,14 @@
         "php": "^7.1.3",
         "ext-ctype": "*",
         "ext-iconv": "*",
+        "symfony/browser-kit": "4.2.*",
         "symfony/console": "4.2.*",
+        "symfony/css-selector": "4.2.*",
         "symfony/dotenv": "4.2.*",
         "symfony/flex": "^1.1",
         "symfony/framework-bundle": "4.2.*",
-        "symfony/test-pack": "^1.0",
+        "symfony/panther": "*",
+        "symfony/phpunit-bridge": "4.2.*",
         "symfony/yaml": "4.2.*"
     },
```

Commits
-------

7f685f6 Copy extra.symfony.require when unpacking
@nicolas-grekas nicolas-grekas deleted the unpack-wildcard branch December 11, 2018 13:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants