wehub-resource-sync
d48cda4081
CI / Test (ubuntu-latest, Node 18.x, bun) (push) Failing after 15m1s
CI / Test (ubuntu-latest, Node 18.x, npm) (push) Failing after 15m1s
CI / Test (ubuntu-latest, Node 18.x, pnpm) (push) Failing after 15m1s
CI / Test (ubuntu-latest, Node 18.x, yarn) (push) Failing after 15m1s
CI / Test (ubuntu-latest, Node 20.x, bun) (push) Failing after 17m13s
CI / Test (ubuntu-latest, Node 20.x, npm) (push) Failing after 18m42s
CI / Test (ubuntu-latest, Node 20.x, pnpm) (push) Failing after 15m0s
CI / Test (ubuntu-latest, Node 20.x, yarn) (push) Failing after 49m44s
CI / Test (ubuntu-latest, Node 22.x, bun) (push) Failing after 51m55s
CI / Test (ubuntu-latest, Node 22.x, pnpm) (push) Failing after 21m57s
CI / Test (ubuntu-latest, Node 22.x, npm) (push) Failing after 37m39s
CI / Test (ubuntu-latest, Node 22.x, yarn) (push) Failing after 34m7s
CI / Validate Components (push) Failing after 37m15s
CI / Python Tests (push) Failing after 10m1s
CI / Security Scan (push) Failing after 10m1s
CI / Lint (push) Failing after 17m12s
CI / Coverage (push) Failing after 20m19s
CI / Test (macos-latest, Node 18.x, bun) (push) Has been cancelled
CI / Test (macos-latest, Node 18.x, npm) (push) Has been cancelled
CI / Test (macos-latest, Node 18.x, pnpm) (push) Has been cancelled
CI / Test (macos-latest, Node 18.x, yarn) (push) Has been cancelled
CI / Test (windows-latest, Node 18.x, npm) (push) Has been cancelled
CI / Test (windows-latest, Node 18.x, pnpm) (push) Has been cancelled
CI / Test (windows-latest, Node 18.x, yarn) (push) Has been cancelled
CI / Test (macos-latest, Node 20.x, bun) (push) Has been cancelled
CI / Test (macos-latest, Node 20.x, npm) (push) Has been cancelled
CI / Test (macos-latest, Node 20.x, pnpm) (push) Has been cancelled
CI / Test (macos-latest, Node 20.x, yarn) (push) Has been cancelled
CI / Test (windows-latest, Node 20.x, npm) (push) Has been cancelled
CI / Test (windows-latest, Node 20.x, pnpm) (push) Has been cancelled
CI / Test (windows-latest, Node 20.x, yarn) (push) Has been cancelled
CI / Test (macos-latest, Node 22.x, bun) (push) Has been cancelled
CI / Test (macos-latest, Node 22.x, npm) (push) Has been cancelled
CI / Test (macos-latest, Node 22.x, pnpm) (push) Has been cancelled
CI / Test (macos-latest, Node 22.x, yarn) (push) Has been cancelled
CI / Test (windows-latest, Node 22.x, npm) (push) Has been cancelled
CI / Test (windows-latest, Node 22.x, pnpm) (push) Has been cancelled
CI / Test (windows-latest, Node 22.x, yarn) (push) Has been cancelled
41 lines
1.1 KiB
Markdown
41 lines
1.1 KiB
Markdown
# Code Review
|
||
|
||
Commit edilmemiş değişikliklerin kapsamlı güvenlik ve kalite incelemesi:
|
||
|
||
1. Değişen dosyaları al: git diff --name-only HEAD
|
||
|
||
2. Her değişen dosya için şunları kontrol et:
|
||
|
||
**Güvenlik Sorunları (KRİTİK):**
|
||
- Hardcode edilmiş kimlik bilgileri, API anahtarları, token'lar
|
||
- SQL injection açıklıkları
|
||
- XSS açıklıkları
|
||
- Eksik input validasyonu
|
||
- Güvenli olmayan bağımlılıklar
|
||
- Path traversal riskleri
|
||
|
||
**Kod Kalitesi (YÜKSEK):**
|
||
- 50 satırdan uzun fonksiyonlar
|
||
- 800 satırdan uzun dosyalar
|
||
- 4 seviyeden fazla iç içe geçme derinliği
|
||
- Eksik hata yönetimi
|
||
- console.log ifadeleri
|
||
- TODO/FIXME yorumları
|
||
- Public API'ler için eksik JSDoc
|
||
|
||
**En İyi Uygulamalar (ORTA):**
|
||
- Mutation desenleri (immutable kullanın)
|
||
- Kod/yorumlarda emoji kullanımı
|
||
- Yeni kod için eksik testler
|
||
- Erişilebilirlik sorunları (a11y)
|
||
|
||
3. Şunları içeren rapor oluştur:
|
||
- Önem derecesi: KRİTİK, YÜKSEK, ORTA, DÜŞÜK
|
||
- Dosya konumu ve satır numaraları
|
||
- Sorun açıklaması
|
||
- Önerilen düzeltme
|
||
|
||
4. KRİTİK veya YÜKSEK sorunlar bulunursa commit'i engelle
|
||
|
||
Güvenlik açıklıkları olan kodu asla onaylamayın!
|