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

php 7 support #4

Closed lekoala closed 6 years ago

lekoala commented 7 years ago

Is there a good reason why php7 is not allowed ? it would be great to allow it since SilverStripe 3.6 now accepts php7 as well

phptek commented 7 years ago

Hi, no. No good reason. I'll look into it. Feel free to test on 7 with 3.6. If tests pass, issue a PR. Thanks!

lekoala commented 7 years ago

i tested with the new sdk (1.7.0) and I've seen no particular issue. if I manage to get some time, i'll send a pr

lekoala commented 7 years ago

https://github.com/phptek/silverstripe-sentry/pull/5

done

phptek commented 6 years ago

Merged ages ago. Closing.