.d.ts files should be forbidden from importing .ts files
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức phù hợp với người mới
- 55/100
- Loại issue
- Tính năng
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- typescript
- Lĩnh vực
- build-system
Hướng nghiên cứu
Trong types/adobe__pdfservices-node-sdk, chạy npm test adobe__pdfservices-node-sdk sau khi đổi tên client-config.d.ts thành client-config.ts như đã chỉ ra. Theo dõi quá trình xác thực xung quanh Index.d.ts và client-config.ts đã được đổi tên, sau đó làm cho bài kiểm tra báo cáo lỗi import mong đợi thay vì lỗi biên dịch hiện tại.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
.d.ts files should not be able to import .ts files; .ts files are supposed to be only for testing. But it's too easy to make a mistake and use .ts for a file intended for shipping types. See for example https://discord.com/channels/508357248330760243/757992230077333655/1086956594736148511
To repro:
cd $dt/types/adobe__pdfservices-node-sdk
git mv client-config.d.ts client-config.ts
npm test adobe__pdfservices-node-sdk
Expected: Error "Index.d.ts can't import from client-config.ts because it's a .ts file"
Actual: Compile error because client-config.ts uses syntax that is illegal in .ts (function signature without declare or following implementation signature).
- Ngôn ngữ chính
- TypeScript
- Star
- 423
- Fork
- 237
- Merge trung bình
- 18 giờ 18 phút
- Pull request đã merge (30 ngày)
- 11
Hướng dẫn đóng góp
Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này
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.
Issue khác của microsoft/DefinitelyTyped-tools
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 66/100
microsoft/DefinitelyTyped-tools#1324 · 7 bình luận ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 32/100
microsoft/DefinitelyTyped-tools#1230 · 1 bình luận ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 42/100
microsoft/DefinitelyTyped-tools#1229 ·
-
microsoft/DefinitelyTyped-tools#1218 · 1 reaction · 1 người được giao ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 45/100
microsoft/DefinitelyTyped-tools#1211 · 1 bình luận ·
Tất cả issue của microsoft/DefinitelyTyped-tools
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
bcgov/bc-wallet-mobile#4761 · 1 bình luận ·
-
external-issue to-triage
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
-
area-deployment area-integrations triage:bot-seen
Độ khó 2/5 Nửa ngày Mức phù hợp với người mới 86/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
-
refactor
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100