WIP: Merge the tiny dockerization work into master #33
Conversation
and fix Dockerfile
… moved to the beginning.
|
Gerekiyor olmasi @haozturk. Su anda testleri yaparken bastan sona temiz kurulum ile ilerlemeli. Aradaki tum puruzleri de gidermeye calisalim. |
|
Evet hocam, modeli open access yaptıktan sonra başka bir PR'da README'yi update'leriz. @uskudarli |
|
@onurgu Selam, phase 1 - phase2 durumu yerine poetry ya da pipenv gibi bi alet daha iyi sonuç verebilir gibi orada. Okay dersek, docker configini ona göre güncelleyebilirim. |
| @@ -0,0 +1,22 @@ | |||
| FROM python:3 | |||
There was a problem hiding this comment.
Projenin spesifik olarak beklediği ya da sorun çıkardığı bir Python sürümü var mı?
There was a problem hiding this comment.
Valla tam emin değilim ama 3 olsun demiştim
|
|
||
| COPY kanarya_config.json /etc/ | ||
|
|
||
| COPY . /game_project/ |
There was a problem hiding this comment.
Bunu komple kopyalamak yerine volume olarak bağlasak, yeniden derlemeden güncelleme şansı olur.
| COPY requirements-phase2.txt /game_project/ | ||
| RUN pip install -r requirements-phase2.txt | ||
|
|
||
| RUN python manage.py migrate --noinput |
There was a problem hiding this comment.
Ben bunu dışarıdan kullanıcıya bırakmayı tercih ediyorum. Otomatik yapınca, yayına alma ya da farklı kullanıcıların farklı branchlerde farklı migrationlarla çalışma durumlarına beklenmedik sonuçlar çıkabiliyor.
haozturk
changed the title
No description provided.