development - Githubissues

devnotcom / devnot-mentor-back-end

Devnot Mentor projesinin Back-end ve Rest API kaynak kodlarını içermektedir.

MIT License

69 stars 21 forks source link

development #30

Closed sabitkondakci closed 3 years ago

sabitkondakci commented 3 years ago

VSCode ile pull işlemi yapıldığında .vscode klasörü oluşuyor bu nedenle .gitignore dosyasına ekleme yaptım
SlugifyParameterTransformator.cs uzun controller isimlerini "-" kullanarak otomatik ayırıyor, uzun metin controller tanımlamalarında işe yarar bir teknik.

yusufyilmazfr commented 3 years ago

PR atıldıktan 30 dakika sonra direkt kapanmış ama ben gene de şunları ekleyeyim.

Öncelikle katkı için teşekkürler. 🖖

.vscode'un her pull sonrası gelmemesi için .gitignore dosyasına eklenmesi mantıklı, SlugifyParameterTransformator gayet kullanışlı görünüyor fakat şu an tanımlanan Route yapılarına baktığım zaman direkt Controller tanımlı bir tanım görülmüyor hepsi manuel elle yazılmış bu yüzden mevcut yapıda durması kodu okuyanlar için bir tık kafa karışıklığına yol açabilir.

Teşekkürler tekrardan.

sabitkondakci commented 3 years ago

Selam Yusuf,

Controller kısmına PR yaptıktan sonra detaylı baktım, isimleri kısa tutmuşsunuz bu nedenle yazdığım kodun gereksiz olduğunu düşündüm ve iptal ettim. Bu arada BenchmarkDotNet kütüphanesini kullanmanızı öneririm, performans analizi için harika araçlar sunuyor, metodun ne kadar bellek tükettiği + garbage collection çalışma detayı + ortalama çalışma süresi gibi birçok parametreyi inceleme şansı sunuyor.

Sağlıcakla.