redhat-developer / redhat-developer/vscode-java

Refactor `activate()` to use `await`

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

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

build/infra debt
Ngôn ngữ chính
TypeScript
Star
2.3k
Fork
546
Merge trung bình
20 giờ 1 phút
Pull request đã merge (30 ngày)
11

Mô tả

Refactor the activate function (main entry point to extension) to use await instead of promise chaining and promise constructors.

Why?

  • The async logic is currently really hard to read
  • The function is > 300 lines of code, and using async could make it shorter
  • The function contains async promise executors, which are considered bad since they make error handling hard

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.

Hướng nghiên cứu

Bắt đầu từ điểm vào activate của extension và xem xét các chuỗi promise cùng các promise constructor của nó. Refactor luồng bất đồng bộ theo hướng mục tiêu await của issue, sau đó chạy các kiểm tra hiện có của project để xác nhận rằng hành vi vẫn không thay đổi.

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, vscode
Lĩnh vực
devtools
Loại issue
Tái cấu trúc
Độ 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
35/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.