microsoft / microsoft/TypeScript
Provide a `this` type for `get`/`set` methods in `Reflect.defineProperty` options
Đang mở
Chưa có ai nhận issue này.
Experience Enhancement
Suggestion
- Ngôn ngữ chính
- Go
- Star
- 111k
- Fork
- 14.3k
- Merge trung bình
- 2 ngày 4 giờ
- Pull request đã merge (30 ngày)
- 132
Mô tả
Code
var A: {a?: number, b: number} = {b:2}
Reflect.defineProperty(A, 'a', {
// set(this: any, value: any) { // to fix
set(value: any) {
console.log(this.b); // Property 'b' does not exist on type 'PropertyDescriptor'.
},
enumerable: true,
configurable: true
});
A.a = 1;
// expect the context of setter is A
Expected behavior:
expect the context of the setter is A
Actual behavior:
Property 'b' does not exist on type 'PropertyDescriptor'.
Playground Link:
Hướng dẫn đóng góp
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
Bắt đầu với ví dụ TypeScript Playground được liên kết và so sánh this được suy luận của setter với kiểu receiver được mong đợi. Truy vết các tùy chọn của Reflect.defineProperty và các khai báo kiểu của PropertyDescriptor, sau đó thêm coverage cho ví dụ vào một khu vực kiểm thử hiện có phù hợp. Được xem là hoàn tất khi ngữ cảnh setter được định kiểu là đối tượng đích mà không có lỗi được báo cáo.
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
- compilers
- Loại issue
- Tính năng
- Độ khó
- 5/5
- Thời gian dự kiến
- Hơn một tuần
- 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
- 25/100