Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

phpunit requirement breaks on Magento 2.2 #105

Open
nesh-younify opened this issue Sep 27, 2017 · 6 comments
Open

phpunit requirement breaks on Magento 2.2 #105

nesh-younify opened this issue Sep 27, 2017 · 6 comments

Comments

@nesh-younify
Copy link

"phpunit/phpunit": "^5.7"

M2.2 (released yesterday) now uses "phpunit/phpunit": "~6.2.0", so composer refuses to install magium/magento due to version conflict. Also seen in https://github.com/magium/Magium/blob/3e7f1d91968ec52b91dba2ffb7e62172ead36819/composer.json#L20

Not sure if it's safe to update phpunit versions on both of them to 6.* so I did not made forks and pull requests (yet).

@kschroeder
Copy link
Collaborator

Sorry for the delay and thank you for bringing it to my attention. I don't currently have it on a schedule to address but I will keep the issue open until I'm done with it. However, if you want to take a stab at it feel free to make a PR.

@JKetelaar
Copy link

I tried upgrading the phpunit version, but now it breaks on symfony/console, as Magento2 requires <= 2.7 and magium/magium requires version 3. :/

@kschroeder
Copy link
Collaborator

@JKetelaar I have updated magium/magium to 1.1.14 which will use either symfony/console 2 or 3. All the tests passed and I was able to install magium/magium, magium/magento and magium/magento2 on Magento 2.2.1

@JKetelaar
Copy link

You, hero!

@kschroeder
Copy link
Collaborator

@nesh-younify I've looked into your issue more and it looks like there are some interface incompatibilities that I will need to take into account for the phpunit version. So, it's still on my radar but it won't be a quick fix.

@kschroeder
Copy link
Collaborator

@nesh-younify Could you try running it with {"magium/magium": "dev-master#e3b4fbabeac84d00b0dff3c3a38b5ec91e7d3bef"}? I've added a compatibility layer (one that I'm not so proud of, but it seems to work) to allow for either PHPUnit 5 or 6.

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

No branches or pull requests

3 participants