phptek / silverstripe-sentry

Flexible Sentry compatible bug aggregation client for Silverstripe applications.
BSD 3-Clause "New" or "Revised" License
12 stars 22 forks source link

allow silverstripe 4.1 #8

Closed lekoala closed 6 years ago

lekoala commented 6 years ago

wououw and the build passes !

lekoala commented 6 years ago

i also remove monolog from the backtrace it's not really necessary

phptek commented 6 years ago

Can you break these up into separate PRs? There are several different categories of fix that arent really related. I'm happy to merge the backtrace and Travis patches, but the composer change will allow any sub version of PHP7, which is not really accurate given that framework itself is only supported up to 7.2 - thanks!

lekoala commented 6 years ago

ok done. i've made another pull requests where composer.json only accepts up to 7.2 and split the other changes into separate pull requests

phptek commented 6 years ago

Closing, in favour of individual PRs as requested.