Comments that apply to a whole file or the whole diff, not a line
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức phù hợp với người mới
- 25/100
- Loại issue
- Tính năng
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- typescript
- Lĩnh vực
- tooling
Hướng nghiên cứu
Start by locating the existing line-comment UI and the import/export paths that serialize anchors. Review the /diffx-finish-review skill documentation as part of the scope. Done means file- and diff-level comments work without changing line comments, older exports still import, and duplicate detection includes scope.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Summary
Every comment is anchored to a line (or line range, #33). Observations about a file as a whole ("this module should be split") or the change as a whole ("missing migration") have no anchor, so reviewers attach them to an arbitrary line.
Proposal
- File-level comment: a "Comment" button in each file header, anchored to the file path only.
- Diff-level comment: a "Comment on diff" button above the first file, anchored to nothing.
- "Copy comments" emits
<comment scope="file">inside the<file>block and<comment scope="diff">outside all<file>blocks. Import parses both and de-duplicates on scope as well as anchor. - Line comments are unchanged on disk and in export (
scopeabsent meansline).
Compatibility
Older exports import unchanged. The /diffx-finish-review skill documentation requires an update for the two new scopes. A PR follows.
- Ngôn ngữ chính
- TypeScript
- Star
- 206
- Fork
- 37
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Chuẩn bị môi trường
Dự án này không cung cấp dev container, Dockerfile hay hướng dẫn đóng góp, nên bạn cần tự thiết lập môi trường: hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- 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.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của wong2/diffx
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 88/100
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 38/100
Issue tương tự
-
area/frontend good first issue kind/cooldown
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
Maintainer thường phản hồi trong vòng 1 ngày
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 85/100
voidzero-dev/oxc-angular-compiler#511 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
langchain-ai/deepagentsjs#898 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 92/100
anomalyco/models.dev#8509 · 2 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug documentation P2 UI/UX
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 85/100
Maintainer thường phản hồi trong vòng 1 ngày