Sylius-Standard icon indicating copy to clipboard operation
Sylius-Standard copied to clipboard

Isolate all containers configuration

Open tonicospinelli opened this issue 3 years ago • 3 comments

it depends #568

tonicospinelli avatar Jun 07 '21 14:06 tonicospinelli

In case of a PHP-FPM error the output of the sylius container should display the error, but it only shows the 500 response code. docker-compose logs -f sylius image

elvetemedve avatar Jul 30 '21 17:07 elvetemedve

Hi @tonicospinelli, can you please elaborate why this change is necessary and what kind issue it's solving? Is it dev/prod feature?

diimpp avatar Nov 23 '23 14:11 diimpp

@diimpp, it has been over two years since I made these changes. The main idea was to isolate vendor and node_modules from the shared folder between the host and the container. It increases the response time for local tests because those folders have thousands of files to be synced and interpreted in runtime.

After this quick change, I saw an opportunity to isolate the configurations and build a process of sylius, php-fpm, node, and other configuration files. Today, reading all the PR to remember that I made, it's evident there must be multiple PRs about my changes.

tonicospinelli avatar Dec 19 '23 13:12 tonicospinelli