Skip to content
This repository has been archived by the owner on Mar 13, 2024. It is now read-only.

Lack of dockerignore file #137

Open
AlexanderWells-diamond opened this issue May 22, 2023 · 2 comments
Open

Lack of dockerignore file #137

AlexanderWells-diamond opened this issue May 22, 2023 · 2 comments

Comments

@AlexanderWells-diamond
Copy link
Collaborator

Currently there is no .dockerignore file. This presents a problem if a user installs their venv into the folder and later tries to use the Dev Container. It results in the container build trying to copy in the entirety of that virtual environment. In my case that copy was running for 10 minutes before I gave up and cancelled it.

A .dockerignore file should be added to cover venv, .venv, and similar folders.

@gilesknap
Copy link
Contributor

I vote for
.venv*
venv*
as I find it useful to put contextual suffixes on venv folders.

@gilesknap
Copy link
Contributor

Um, or it should match .gitignore which is the same as the above

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants