search

Grsaiago / ft_transcendence

Transcending in django
0 stars 0 forks source link

Add: Configs for compose and postgres #3

Closed Grsaiago closed 2 months ago

Grsaiago commented 2 months ago

Oi, gente! Subi um compose com postgres e redis e coloquei as confs pra gente usar o postgres como banco de dados! Vou colocar o .env anexo aqui, mas qqr coisa só me dar um alô <3 me-renomeie-para-.env-e-tire-a-extenssao.csv

izacabral commented 2 months ago

Fala ae!!! Tentei rodar a branch para validar, mas tive alguns bloqueios... quando tu puder avisa pra gente trocar sobre isso.

Outra coisa, tu fez PR pra main, é isso mesmo?

Grsaiago commented 2 months ago

Fala ae!!! Tentei rodar a branch para validar, mas tive alguns bloqueios... quando tu puder avisa pra gente trocar sobre isso.

Hoje to livre praticamente a tarde toda, quando puder é só me dar um alo no wpp ou disc que a gente vê esses lances que deram erro!

Outra coisa, tu fez PR pra main, é isso mesmo?

talvez...

izacabral commented 2 months ago

Fala ae!!! Tentei rodar a branch para validar, mas tive alguns bloqueios... quando tu puder avisa pra gente trocar sobre isso.

Hoje to livre praticamente a tarde toda, quando puder é só me dar um alo no wpp ou disc que a gente vê esses lances que deram erro!

Sobre o erro como vimos foi causado apenas porque usei docker-compose up com hifen, que é uma versão antiga. O correto é usar docker compose up

Outra coisa, tu fez PR pra main, é isso mesmo?

talvez...

Acho uma boa prática mandar sempre pra develop, se for uma coisa grande e fechada como o DB a gente manda da dev para Main tbm. Pq se não a dev ficaria sem essa atualização... e ao fazer novas branchs a gente faz da dev...

Grsaiago commented 2 months ago

Fala ae!!! Tentei rodar a branch para validar, mas tive alguns bloqueios... quando tu puder avisa pra gente trocar sobre isso.

Hoje to livre praticamente a tarde toda, quando puder é só me dar um alo no wpp ou disc que a gente vê esses lances que deram erro!

Sobre o erro como vimos foi causado apenas porque usei docker-compose up com hifen, que é uma versão antiga. O correto é usar docker compose up

Outra coisa, tu fez PR pra main, é isso mesmo?

talvez...

Acho uma boa prática mandar sempre pra develop, se for uma coisa grande e fechada como o DB a gente manda da dev para Main tbm. Pq se não a dev ficaria sem essa atualização... e ao fazer novas branchs a gente faz da dev...

Obrigado pelo feedback, Iza! Mudei o target desse merge para a develop. Posso prosseguir com o Merge?