microsoft / microsoft/TypeScript
Only prompt the first matching definition
Đang mở
Chưa có ai nhận issue này.
Needs Investigation
- 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ả
Does this issue occur when all extensions are disabled?: Yes
- VS Code Version: Version: 1.58.0
- OS Version: macos catalina 10.15.7
Steps to Reproduce:
// package 'A'
export interface IConfig {}
export class Define {
public config: IConfig
}
// -------------------------- //
// package 'B'
// package 'B' - a.ts
declare module 'A' {
interface IConfig {
setB1: () => {}
}
}
// package 'B' - b.ts
declare module 'A' {
interface IConfig {
setB2: () => {}
}
}
// -------------------------- //
// package 'C'
declare module 'A' {
interface IConfig {
setC: () => {}
}
}
// -------------------------- //
// use
import { IConfig, Define } from 'A';
const a: IConfig // no problem, support `setB1` `setB2` `setC`
const b = new Define();
b.config // error, only prompt support for `setB1` method
Is there a way to make b.config prompt for all methods? In addition to using b.config as Iconfig
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
Tái hiện ví dụ về gộp khai báo từ issue với các package A, B và C, sau đó so sánh tính năng hoàn tất cho một giá trị IConfig trực tiếp với tính năng hoàn tất cho b.config. Truy vết hành vi liên quan của TypeScript language-service hoặc hoạt động kiểm tra kiểu, và thêm một bài kiểm thử hồi quy cho thấy tất cả các member đã được gộp đều được cung cấp cho b.config.
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ó
- 3/5
- Thời gian dự kiến
- 1-2 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