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
don't install unneeded polyfills with composer #16653
Conversation
That package actually was already automatically removed with a composer update. See #16649 |
Ah, I didn't see that PR. |
Yes. Shouldn't hurt to prevent installing those polyfills nevertheless |
@Findus23 @sgiehl the tests are failing see https://travis-ci.com/github/matomo-org/matomo/jobs/424314293 |
guess merging in the latest changes should fix it... |
there's a test now failing. not sure if it is due to this? https://travis-ci.com/github/matomo-org/matomo/jobs/424766921#L925 @sgiehl @Findus23 |
The same failure seems to be on 4.x-dev without this PR: https://travis-ci.com/github/matomo-org/matomo/jobs/424766869#L921 |
That was caused by the cookie PR. See #14444 (review) |
see https://github.com/symfony/polyfill/blob/main/README.md#design
This removes
symfony/polyfill-php56
on the nextcomposer install