microsoft / microsoft/TypeScript
Incomprehensible incompatibility around Partial, Conditional types and Generics.
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- Go
- Star
- 111k
- Fork
- 14.4k
- Merge trung bình
- 1 ngày 19 giờ
- Pull request đã merge (30 ngày)
- 117
Mô tả
TypeScript Version: 3.9.0-dev.20200322
Search Terms: partial, conditional types
Code
type SafePartial<T> = T extends {} ? Partial<T> : any;
interface QB<TRecord extends {}> {
insert(record: SafePartial<TRecord>): void;
}
async function insert1<TRecord extends {}>(qb: QB<TRecord>, record: TRecord) {
await qb.insert(record);
}
Expected behavior:
The code successfully compiles
Actual behavior:
Following error is observed for qb.insert(record):
Argument of type 'TRecord' is not assignable to parameter of type 'SafePartial<TRecord>'. Type '{}' is not assignable to type 'SafePartial<TRecord>'.
The issue goes away if I use Partial instead of the SafePartial above.
The issue also goes away if a concrete type is used instead of generic type:
interface User {
id: string;
}
async function insert1(qb: QB<User>, record: User) {
await qb.insert(record);
}
Playground Link: Playground
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 TypeScript Playground được liên kết và so sánh trường hợp kiểu điều kiện generic với các trường hợp Partial và kiểu cụ thể đang hoạt động. Theo dõi cách compiler xử lý khả năng gán cho SafePartial và xác nhận hành vi bằng một bản tái hiện rút gọn. Hoàn thành khi mã gốc biên dịch được mà không làm yếu các kiểu đã 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
- 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
- Đặc tả rõ ràng
- Mức phù hợp với người mới
- 42/100