microsoft / microsoft/vscode-pull-request-github

Comments attached to wrong lines

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

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

info-needed
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:

  1. Install the "GitHub Pull Requests" extension to VS Code.
  2. View the affected PR in VS Code.
  3. Search for a string from the comment to see it in VS Code

The right placement when viewed with Chromium:
Image

The wrong placement when viewed in VS Code:
Image

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

  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.

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

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.