Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

Investigate local code review tooling to reduce PR review iterations

Đang mở
  • Có gì không ổn?
#35 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 2 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
5/5
Thời gian dự kiến
Hơn một tuần
Mức phù hợp với người mới
35/100
Loại issue
Tính năng
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Ít trao đổi
Công nghệ
csharp, github
Lĩnh vực
developer-experience, tooling

Hướng nghiên cứu

Bắt đầu bằng việc xem xét PR #33 và 28 nhận xét của nó để xác định ít nhất ba loại issue cần tái hiện. So sánh gh copilot CLI, Copilot code review API, một script pre-push tùy chỉnh và các trình phân tích Roslyn, bao gồm cả cấu hình StyleCop và Roslynator hiện có. Được xem là hoàn tất khi đã đánh giá ít nhất hai tùy chọn, triển khai tùy chọn thực tế nhất và chứng minh rằng tùy chọn đó phát hiện được ba loại issue của PR #33.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

enhancement

Summary

Investigate options for running automated code review locally before pushing, to reduce the number of PR review iterations.

Background

PR #33 went through 8 rounds of Copilot review with 28 total comments. Many issues (edge cases, naming, platform concerns) could have been caught before the first push if a local review step existed.

Options to investigate

  1. gh copilot CLI — The general-purpose Copilot CLI (gh copilot) exists but has no review subcommand. Investigate if it can be prompted to review a diff or staged files for code quality.

  2. GitHub Copilot code review API — Check if Copilot pull request reviewer can be invoked programmatically on a local diff before creating a PR.

  3. Custom pre-push script — A script that runs the diff through a local LLM or the Copilot API with the repo's coding guidelines, then blocks the push if issues are found.

  4. Roslyn analyzers — We already have StyleCop and Roslynator. Consider adding:

    • Custom analyzers for single-letter lambda detection
    • Custom analyzers for ConfigureAwait(false) without named parameter
    • These would catch issues at build time, not just in pre-commit grep

Acceptance criteria

  • Evaluate at least 2 options above
  • Implement the most practical option
  • Demonstrate it catches at least 3 of the issue types from PR #33
Ngôn ngữ chính
C#
Star
0
Fork
4
Merge trung bình
2 ngày 8 giờ
Pull request đã merge (30 ngày)
7

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của Azure/Connectors-NET-LSP

Tất cả issue của Azure/Connectors-NET-LSP

Issue tương tự

Thêm issue về C#

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.