Skip to content

Commit 917c289

Browse files
authored
add --rm to docker compose run commands
1 parent 9a9a4cf commit 917c289

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

README.rst

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -35,8 +35,8 @@ Try it
3535
cd django-cms-quickstart
3636
docker compose build web
3737
docker compose up -d database_default
38-
docker compose run web python manage.py migrate
39-
docker compose run web python manage.py createsuperuser
38+
docker compose run --rm web python manage.py migrate
39+
docker compose run --rm web python manage.py createsuperuser
4040
docker compose up -d
4141
4242
Then open http://django-cms-quickstart.127.0.0.1.nip.io:8000 (or just http://127.0.0.1:8000) in your browser.
@@ -77,7 +77,7 @@ Updating requirements
7777
The project uses a django best practise two step approach, freezing all dependencies with pip-tools. Here is how to update requirements:
7878

7979
1. Change `requirements.in` according to your needs. There is no need to pin the package versions here unless you have a good reason (i.e. known incompatibilities)
80-
2. Run `docker compose run web pip-compile requirements.in >> requirements.txt`
80+
2. Run `docker compose run --rm web pip-compile requirements.in >> requirements.txt`
8181
3. `requirements.txt` should now have changed
8282
4. Rebuild the container `docker compose build web` and restart `docker compose up -d`
8383

0 commit comments

Comments
 (0)