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

Never change memory_limit = -1 or max_execution_time = 0 #36

Merged
merged 2 commits into from
Mar 21, 2017

Conversation

AntonEvers
Copy link
Contributor

Fixes magento/magento2#2461
Replaces #28
Replaces #31

@AntonEvers
Copy link
Contributor Author

Hello @tubalmartin,

It would be greatly appreciated if you would approve this pull request soon, as the https://github.com/magento/magento2 project depends on it. You would help the Magento 2 community a great deal.

Fixes magento/magento2#2461

Thank you!

@AntonEvers
Copy link
Contributor Author

I have tested this in a Magento 2 project that consistently produced errors because of this bug. Now it runs without errors.

@tubalmartin
Copy link
Owner

Thank you for the MR!

@AntonEvers
Copy link
Contributor Author

Whow, that is FAST! Thanks @tubalmartin

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