Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Validate .env variables before using inside config services #177

Open
CllDMR opened this issue Feb 16, 2023 · 9 comments
Open

Validate .env variables before using inside config services #177

CllDMR opened this issue Feb 16, 2023 · 9 comments
Assignees

Comments

@CllDMR
Copy link
Collaborator

CllDMR commented Feb 16, 2023

.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
Copy link

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

@muraterzioglu
Copy link

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

@CllDMR
Copy link
Collaborator Author

CllDMR commented Feb 16, 2023

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
Copy link
Collaborator Author

CllDMR commented Feb 16, 2023

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

@muraterzioglu
Copy link

muraterzioglu commented Feb 16, 2023

@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
Copy link
Collaborator Author

CllDMR commented Feb 16, 2023

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
Copy link

Tamamdır teşekkür ederim!

@busebatman
Copy link
Collaborator

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
Copy link
Collaborator Author

CllDMR commented Feb 16, 2023

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

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

No branches or pull requests

3 participants