microsoft / microsoft/TypeScript
JS output contains imports only used in types when `emitDecoratorMetadata: true`
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ả
Bug Report
🔎 Search Terms
- emitDecoratorMetadata require
- emitDecoratorMetadata import
🕗 Version & Regression Information
- This is the behavior in every version I tried, and I reviewed the FAQ for entries about 3.3.3, 4.1.3
⏯ Playground Link
Playground link with relevant code
💻 Code
import * as vscode from 'vscode';
function LogMethod(
target: any,
propertyKey: string | symbol,
descriptor: PropertyDescriptor
) {
console.log(target);
console.log(propertyKey);
console.log(descriptor);
}
class Demo {
@LogMethod
public foo(bar: vscode.Memento) {
// do nothing
}
}
const demo = new Demo();
🙁 Actual behavior
The output contains const vscode = __importStar(require("vscode")); when emitDecoratorMetadata: true but not when emitDecoratorMetadata: false
🙂 Expected behavior
As the type of bar as nothing to do with the compiled JS, the import of vscode should not exist regardless of the value of emitDecoratorMetadata
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 JavaScript được tạo ra khi emitDecoratorMetadata được bật và tắt. Báo cáo không nêu tên tệp hoặc test nào trong repository, vì vậy việc xác định đường dẫn của compiler sẽ cần đọc phần triển khai liên quan. Được coi là hoàn tất khi import type-only của vscode không xuất hiện trong cả hai output.
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
- 45/100