Skip to content

Conversation

@ram-fabian-heinrich
Copy link
Contributor

@ram-fabian-heinrich ram-fabian-heinrich commented Sep 4, 2023

Based on #127:

  • Install all application dependencies in the same block

  • Add --no-install-recommends switch to reduce image size

  • Install all PHP extensions in the same block

  • Add version to DB base image to fix Dockerfilelint issue

Additional changes:

  • Removed the copyiny of files to /var/www/html. Added a volume instead. This volumes syncs local files with files in docker container.
  • Configuration files must now be created manually. Added step to CONTRIBUTING.md.
  • Added xdebug extension.
  • Docker php version from 7.2 to 8.1.

@ram-fabian-heinrich ram-fabian-heinrich changed the title Optimized dockerfiles Optimized dockerfiles - PHP 8.1, volume and xdebug Sep 4, 2023
@anuko
Copy link
Owner

anuko commented Sep 14, 2023

Hello, it may take me a while to review this due to other emergencies.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants