microsoft / microsoft/TypeScript
No excess property error for computed property
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- Go
- Star
- 111k
- Fork
- 14.3k
- Merge trung bình
- 1 ngày 19 giờ
- Pull request đã merge (30 ngày)
- 117
Mô tả
TypeScript Version: 3.7.5 and 3.9.0-dev.20200220
Search Terms: excess property, computed property, index signature
Code
interface Foo {
a: string;
}
const bar: Foo = {
a: "",
z: "" // error, excess prop
}
const baz: Foo = {
a: "",
["Z".toLowerCase()]: "" // this is okay I guess?
}
Expected behavior:
The computed property in the definition of baz should maybe produce an excess property error, since Foo has no string index signature and "Z".toLowerCase() is only known to be a string.
Actual behavior:
The computed property is allowed.
Playground Link: Here
Related Issues: #22427
This issue is a re-opening of #22427, which was marked as a bug and closed as "fixed", but I don't see an actual fix referenced anywhere in that issue and the behavior described in that issue persists. Any ideas? Thanks!
There's an SO question asking this, which brought me to #22427, which has me scratching my head 😕.
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 bản tái hiện trong TypeScript Playground ở issue và so sánh trường hợp thuộc tính được tính toán với trường hợp thuộc tính dư thừa trực tiếp. Đọc issue liên quan #22427 để hiểu cuộc thảo luận trước đó; công việc được xem là hoàn tất khi hành vi thuộc tính dư thừa mong đợi được giải quyết hoặc hành vi dự định được ghi lại bằng các bài kiểm thử compiler đã cập nhật.
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
- 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
- 35/100