-
Notifications
You must be signed in to change notification settings - Fork 47
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
1.0.6 and MageRun2 #122
Comments
@fballiano Currently we skip the compatibility check in Magerun. In the future I can add a logic in Magerun if something is incompatible. The interesting code in Magerun is here: https://github.com/netz98/n98-magerun2/blob/8636430a8a990d581c1eeb7137b7681f79afd41b/src/N98/Magento/Application/Console/EventSubscriber/CheckCompatibility.php#L92 |
ok, we need a smaller change due to the new |
@fballiano I have merged some changes to the branch for the next n98-magerun2 release. If everything works then you should see no compatibility issue anymore and the |
tested and perfectly working, thank you so much @cmuench ![]() |
Everything is explained in this comment:
#115 (comment)
I'm opening this issue just not to forget about it.
The text was updated successfully, but these errors were encountered: