microsoft / microsoft/vscode-pull-request-github
Comments attached to wrong lines
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- TypeScript
- Star
- 2.6k
- Fork
- 796
- Merge trung bình
- 1 ngày 4 giờ
- Pull request đã merge (30 ngày)
- 46
Mô tả
- Extension version: 0.124.1
- VSCode Version: 1.107.1
- OS: Fedora 42
- Repository Clone Configuration (single repository/fork of an upstream repository): fork of an upstream repo
- GitHub Product (GitHub.com/GitHub Enterprise version x.x.x): GitHub.com
See here for the affected PR and the comment: https://github.com/scylladb/scylladb/pull/27386/files#r2643211612
When viewing with a browser, the comment is in the right place, right where there are 7 lines removed and the one below:
for shard in iterate_over_describe_stream(dynamodbstreams, arn, time.time() + 60):
is added instead.
In VS Code the comment is shown associated to the wrong lines - to the new lines starting from 1409, not the old lines starting from 1409.
Steps to Reproduce:
- Install the "GitHub Pull Requests" extension to VS Code.
- View the affected PR in VS Code.
- Search for a string from the comment to see it in VS Code
The right placement when viewed with Chromium:
The wrong placement when viewed in VS Code:
Hướng dẫn đóng góp
Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này
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.
Hướng nghiên cứu
Tái hiện bình luận bị đặt sai vị trí trong extension GitHub Pull Requests bằng cách mở PR #27386 trong VS Code và tìm văn bản của bình luận. So sánh vị trí của extension với chế độ xem Chromium, và coi issue là hoàn tất khi bình luận được gắn với các dòng đã bị xóa thay vì các dòng mới bắt đầu từ 1409.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- typescript
- Lĩnh vực
- devtools
- Loại issue
- Lỗi
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Khá rõ ràng
- Mức phù hợp với người mới
- 45/100