Peer Review

📝 Peer Review

Akademik ve teknik peer review metodolojisi rehberi.

📋 Review Framework

Değerlendirme Alanları

🔍 Code Review Checklist

## Functionality
- [ ] Kod beklendiği gibi çalışıyor mu?
- [ ] Edge case'ler handle ediliyor mu?
- [ ] Error handling yeterli mi?

## Code Quality
- [ ] DRY prensibi uygulanmış mı?
- [ ] Naming convention tutarlı mı?
- [ ] Comments yeterli mi?

## Security
- [ ] Input validation var mı?
- [ ] SQL injection riski var mı?
- [ ] Sensitive data korumalı mı?

## Performance
- [ ] Gereksiz işlem var mı?
- [ ] Memory leak riski var mı?

📄 Document Review Template

## Review Summary
**Document:** [Doküman adı]
**Reviewer:** [İsim]
**Date:** [Tarih]

## Overall Assessment
[Genel değerlendirme - 1-2 paragraf]

## Strengths
1. ...
2. ...

## Areas for Improvement
1. ...
2. ...

## Specific Comments
| Section | Comment | Severity |
|---------|---------|----------|
| ... | ... | Major/Minor |

## Recommendation
[ ] Accept
[ ] Minor Revisions
[ ] Major Revisions
[ ] Reject

💬 Constructive Feedback

İyi Feedback

✅ "Bu fonksiyon X durumunda hata verebilir. 
    Try-catch eklemeyi düşünebilir misin?"

✅ "Güzel implementasyon! Bir öneri: 
    Bu method extract edilse daha okunabilir olur."

Kaçınılması Gereken

❌ "Bu yanlış"
❌ "Neden böyle yaptın?"
❌ "Ben olsam böyle yapmazdım"

Peer Review v1.1 - Enhanced

🔄 Workflow

Kaynak: Conventional Comments & Google Engineering Practices

Aşama 1: Preparation (Reviewee)

• Self-Review: Kodu pushlamadan önce kendin oku, gereksiz logları temizle.
• Context: PR açıklamasında "Ne?", "Neden?" ve "Nasıl Test Edilir?" sorularını yanıtla.
• Scope: Değişikliği yönetilebilir boyutta tut (<400 satır tercihen).

Aşama 2: Review Process (Reviewer)

• Clarity: Kod ne yaptığını anlatıyor mu? Değişken isimleri açıklayıcı mı?
• Security: Kullanıcı girdisi sanitize ediliyor mu? Auth kontrolü var mı?
• Performance: Gereksiz döngüler veya N+1 sorguları var mı?
• Tone: Yorumlarda nit:, suggestion:, blocking: gibi etiketler kullan (Conventional Comments). "Sen" dili yerine "Kod" dili kullan.

Aşama 3: Approval & Merge

• CI Checks: Tüm testler ve lint kontrolleri yeşil mi?
• Resolution: Tüm kritik yorumlar (blocking) çözüldü mü?
• Squash & Merge: History temizliği için commitleri birleştir.

Kontrol Noktaları