microsoft / microsoft/TypeScript
Make ParameterDecorator and the like generic
Chưa có ai nhận issue này.
- 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ả
lib Update Request
Missing / Incorrect Definition
ParameterDecorator is currently not generic, it can be attachted to any parameter. However, when a decorator is used for e.g. Dependency Injection purposes, some decorators require the type of the parameter to be consistent with the type of the decorator
Sample Code
E.g. we want this to work
class MyController {
requestThings(@UserId() userId: number): Thing[] { return [] }
}
But this to fail:
class MyController {
requestThings(@UserId() userId: string): Thing[] { return [] }
}
(as userId will be a number, not a string so the injection should ideally not be possible (at compile time)
Currently UserId is done more or less this way:
const UserId = (): ParameterDecorator => (target, key, index) => {
// do stuff
}
while I'd like to add number to ParameterDecorator:
const UserId = (): ParameterDecorator<number> => (target, key, index) => {
// do stuff
}
and I'd like typescript to check this when applying a decorator.
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 bằng cách xác định định nghĩa của ParameterDecorator và các kiểm tra của compiler đối với việc áp dụng các decorator tham số. So sánh các ví dụ về number và string trong issue với hành vi hiện tại, sau đó xác định cách các kiểu tham số generic nên được biểu diễn và xác thực. Công việc được xem là hoàn tất khi các trường hợp áp dụng decorator hợp lệ được chấp nhận và các kiểu tham số không khớp bị từ chối bằng các test phù hợp.
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
- 30/100