@PatOnTheBack opened this Pull Request on July 13th 2019 Contributor

Automated fix by dependabot.

@Findus23 commented on July 13th 2019 Member

This isn't that important as this directory isn't included in the released Matomo build, but just used for tests.
Instead all those dependencies should be updated sometime at once: #14549

@PatOnTheBack commented on July 13th 2019 Contributor

@Findus23 Would it make sense to just merge this PR now and gradually update dependencies?

@PatOnTheBack commented on July 13th 2019 Contributor

The CI test failed for PHP 5, not PHP 7.

@sgiehl commented on July 15th 2019 Member

The library you updated is only used for screenshot testing, which is the build running on PHP 5 only. But that shouldn't be caused by this update. So guess it wouldn't harm to merge that :man_shrugging:

This Pull Request was closed on July 15th 2019
Powered by GitHub Issue Mirror