reviewing-prs

tarafından flutter

gh CLI kullanarak GitHub Pull Request'lerini getirme, inceleme, gözden geçirme, kullanıcıyla uyumlu inceleme yorumları taslağı hazırlama ve onay güvence altına alma için genel iş akışı…

npx skills add https://github.com/flutter/devtools --skill reviewing-prs

Reviewing Pull Requests

This skill outlines the workflow for inspecting GitHub Pull Requests using the gh CLI, drafting review feedback, and securing user approval before posting review comments.

Approval Safeguard (Strict Requirement)

[!IMPORTANT] NEVER post comments or reviews to GitHub without explicit prior user approval. Always present draft review comments to the user in natural language first. Only execute write commands (gh pr comment, gh pr review) after the user approves.

Workflow

1. Request Information via GitHub CLI

  • PR Details:
    gh pr view <pr-number> --repo <owner/repo> --json title,body,author,state,headRefName,baseRefName,comments,reviews,files
    
  • Code Diff:
    gh pr diff <pr-number> --repo <owner/repo>
    
  • Existing Inline Comments:
    gh api repos/<owner/repo>/pulls/<pr-number>/comments
    
  • CI / Status Checks:
    gh pr checks <pr-number> --repo <owner/repo>
    

2. Inspect Context & Prior Feedback

  • Read the PR description, linked issues, and full diff.
  • Verify whether existing bot or human comments have already been addressed in subsequent commits.

3. Draft Review Comments

  • Keep comments direct, concise, and focused on code quality and correctness.
  • Approvals: Keep comments concise (LGTM or A couple comments but lgtm.). Avoid fluffy praise or re-summarizing the PR.
  • Actionable Feedback: Reference specific files, line numbers, and rationale when leaving suggestions.

4. Present Draft & Post Only Upon User Approval

[!IMPORTANT] NEVER post comments or reviews to GitHub without explicit prior user approval.

  1. Present the drafted review comments to the user in your response window.
  2. Ask for confirmation: "Would you like me to submit this review to GitHub?"
  3. Once explicitly approved by the user, post the review:
    gh pr review <pr-number> --repo <owner/repo> --comment --body "<approved review text>"
    

flutter tarafından daha fazla skill

dart-modern-features
flutter
Modernizasyon için adayları bulmak amacıyla:
flutter-fix-layout-issues
flutter
Flutter düzen hatalarını (taşmalar, sınırsız kısıtlamalar) Dart ve Flutter MCP araçlarını kullanarak düzeltir. "RenderFlex overflowed", "Vertical…" sorunlarını giderirken kullanın.
adding-release-notes
flutter
Kullanıcıya yönelik değişiklik açıklamalarını DevTools sürüm notlarına ekler. NEXT_RELEASE_NOTES.md dosyasında iyileştirmeleri, düzeltmeleri veya yeni özellikleri belgelemek için kullanın.
reviewing-devtools-prs
flutter
DevTools deposuna özgü, DevTools stil kurallarını ve yaygın inceleme desenlerini uygulayan PR inceleme iş akışı. … içindeki pull request'leri incelerken kullanın.
dart-use-primary-constructors
flutter
Kullanıcıların Dart'ta sözdizimsel ve anlamsal olarak doğru birincil constructor'lar yazmasına yardımcı olun ve yeni constructor sözdizimine, boş gövdeli noktalı virgül sözdizimine geçiş yapın/kullanın,…
code-documentation
flutter
Etkili kod dokümantasyonu yazma rehberi; docstring'ler, JSDoc, dartdoc ve uygulama yorumları dahil. Yeni kod yazarken, eklerken…
api-review
flutter
Belirtilen kodu, kanonik API Tasarım yönergelerine göre inceler. Kullanıcı bir API incelemesi istediğinde veya kodun API tasarımına uygunluğunu kontrol etmek istediğinde bu beceriyi kullanın…
flutter-accessibility
flutter
Flutter uygulamalarında WCAG 2 ve EN 301 549 erişilebilirlik standartları ile uyarlanabilir düzenleri uygular. Mobil, web ve masaüstü platformlarında anlamsal açıklamaları, dokunma hedef boyutlarını (minimum 48x48 dp) ve metin kontrast oranlarını (küçük metin için 4.5:1, büyük metin için 3:1) zorunlu kılar. Web anlambilimi başlatma, etkileşimli widget sarma, ekran boyutuna dayalı düzen değiştirme ve klavye/fare giriş işleme için karar mantığı sağlar. FocusTraversalGroup ve... aracılığıyla odak geçiş yönetimini içerir.