Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Optimized dockerfiles - PHP 8.1, volume and xdebug #136

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

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