A better SubmoduleStatus?

Đang mở
#1,121 4 bình luận 0 reaction 0 người được giao Xem trên GitHub

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
20/100
Loại issue
Tái cấu trúc
Độ rõ ràng
Cần làm rõ
Mức độ hoạt động
Đình trệ
Công nghệ
csharp, git
Lĩnh vực
backend-api-design

Hướng nghiên cứu

Bắt đầu với cuộc thảo luận được liên kết trong issue 1103 và kiểm tra các API hiện có Submodule.RetrieveStatus(), SubmoduleCollection và SubmoduleStatus. Issue không nêu tên tệp hoặc test nào. Để được xem là hoàn tất, cần có một thiết kế lại API được thống nhất, bao quát việc sở hữu trạng thái, các thuộc tính shortcut và các cờ trạng thái trong tương lai.

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

Mô tả

Stabilization

As discussed in https://github.com/libgit2/libgit2sharp/issues/1103#issuecomment-113903882

Maybe should we

  • Move Submodule.RetrieveStatus() into SubmoduleCollection.RetrieveSubmoduleStatus()
  • Turn SubmoduleStatus into a concrete type, which would expose IsUnmodified and IsWorkingDirectoryDirty shortcut properties along with another one which would maybe expose the flags? This may leave us some room for future improvement (conflicts, ...)

Quote from @dahlbyk

"Now that you mention it, I agree that the SubmoduleStatus API has some room for improvement (SRP violation, communicating both superproject and submodule status)."

Ngôn ngữ chính
C#
Star
3.5k
Fork
925
Chỉ số merge pull request
Không có pull request nào được merge trong 30 ngày

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

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 libgit2/libgit2sharp

Tất cả issue của libgit2/libgit2sharp

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.