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

Only include actually needed things in the docker image instead of the whole repository folder. #101

Open
1 task
Wabuo opened this issue Jan 31, 2024 · 0 comments
Labels
enhancement New feature or request
Milestone

Comments

@Wabuo
Copy link

Wabuo commented Jan 31, 2024

Describe your issue

I don't see any reason why it would be a good idea to just copy the whole repository folder in to the docker image instead of only the for production required files.

It's definitely against docker convention.

Device and settings

docker

Steps to reproduce

From the docker file

# copy application files
COPY . /var/www/localhost/htdocs/

This should be changed to only copy the needed things.

Screenshots (optional)

No response

Extra fields

  • I'd like to work on this issue
@Wabuo Wabuo added the bug Something isn't working label Jan 31, 2024
@Zavy86 Zavy86 added enhancement New feature or request and removed bug Something isn't working labels Aug 23, 2024
@Zavy86 Zavy86 added this to the Release 1.1 milestone Aug 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants