search

lorisleiva / laravel-docker

🐳 Generic docker image for Laravel Applications
MIT License
927 stars 313 forks source link

add calendar php extension #45

Closed ragingdave closed 4 years ago

ragingdave commented 4 years ago

fixes #44

lorisleiva commented 4 years ago

Thanks! 👍

Just wondering, do we not need to also add docker-php-ext-configure calendar?

ragingdave commented 4 years ago

@lorisleiva I left it as I PR'd because I actually had a docker image forked from your repo created first that had calendar and it seemed to be working fine with just the part I committed....so no?

lorisleiva commented 4 years ago

Haha okay no worries. If it works that’s fine. :)