acikyazilimagi / afetlojistik-backend

Apache License 2.0
12 stars 0 forks source link

Validate .env variables before using inside config services #177

Open CllDMR opened 1 year ago

CllDMR commented 1 year ago

.env değerlerinin eksik olması, yanlış formatta olması bu backend uygulamasının runtime'ında hata oluşturmasına neden olabilir. Bu değerler henüz uygulama initialize edilir iken kontrolden geçirilebilir. Bu sayede app ayağa kalmadan env değerlerindeki hataları yakalayabiliriz. Bu konuyu implement ederken Nestjs dokümantasyonu yardımcı olacaktır.

muraterzioglu commented 1 year ago

Merhabalar, bunun üzerine yardımcı olabilirim. app.module.ts ve configuration.ts dosyaları bütün konfigrasyonu içeriyor değil mi?

muraterzioglu commented 1 year ago

@CllDMR bu issue ile bearber #178 için de çalışabilir aynı zamanda Dockerfile'a da .env ile ilgili eklemeler yapmak isterim

CllDMR commented 1 year ago

Merhabalar, evet #178 ile yakın konular. Dockerfile için ayrı bir issue açıp oradan takip edebiliriz. Issue'ları olabildiğince küçük tutup; kolay açıklanabilir, az efor gerektiren şekilde tutmak daha sağlıklı olabilir.

CllDMR commented 1 year ago

Issue'yu sana assign etmek için bende şu an yeterli yetki yok. Yetkisi olan biri bulup isteğini iletebilirim.

muraterzioglu commented 1 year ago

@CllDMR Çok sevinirim, docker için öneri şeklinde de yeni bir issue açabilirim 👍 Doğrudan projeyi pull edip, yeni branch açıp oradan pull request açıyorum değil mi?

CllDMR commented 1 year ago

Evet, şuan için main branch'te iken pull edip, daha sonra main branch'i kullanarak yeni branch oluşturabilirsin. Branch isimlendirmeleri ve commitlint hakkında şuan belirlenmiş bir format yok.

muraterzioglu commented 1 year ago

Tamamdır teşekkür ederim!

busebatman commented 1 year ago

Issue'yu sana assign etmek için bende şu an yeterli yetki yok. Yetkisi olan biri bulup isteğini iletebilirim.

@yldrmzffr bilgin var mı?

CllDMR commented 1 year ago

Benim önerim branching olarak trunk-based strategy, commit'lerde de commitlint formatıdır. Ancak bunlar başka issue'ların konusu :)