Enable assertion exceptions for PHP 7.0 and later

liip / drifter

Drifter is a framework to help provision developer boxes using Ansible and Vagrant

https://liip-drifter.readthedocs.io/

Do What The F*ck You Want To Public License

57 stars 28 forks source link

Closed mjanser closed 5 years ago

mjanser commented 5 years ago

Assertions should be enabled in development mode. Since PHP 7.0 assertions can throw exceptions to make them more visible.

This PR is a: Improvement

chregu commented 5 years ago

Mmmh, couldn't that trigger BC issues for some projects, when it suddenly now throws exceptions?

mjanser commented 5 years ago

Yes, it could actually trigger BC issues. I changed it to be an option now, which is disabled by default.