`cli/common/tools.js` and `tools.cjs` are not covered by `npm run typecheck`
Maintainer thường phản hồi trong vòng 1 ngày
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 3/5
- Thời gian dự kiến
- Nửa ngày
- Mức phù hợp với người mới
- 72/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức độ hoạt động
- Sôi nổi
- Công nghệ
- javascript, typescript
- Lĩnh vực
- build-system
Hướng nghiên cứu
The gap is in the include array of tsconfig.json, which lists "cli/*" and so misses the nested cli/common/ directory. Start by running npm run typecheck and confirming that cli/common/tools.js and tools.cjs are absent from the checked files, then compare with bins.cjs, which is checked only because bins.js imports it. Done means the typecheck covers both files and either passes or surfaces type errors that are then fixed.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Found by differential semantic drift analysis from ece8b148 to a7721303.
What is broken
cli/common/tools.js and tools.cjs, added by #8694, start with // @ts-check, but tsconfig.json includes "cli/*", which does not reach cli/common/. bins.cjs is checked only because bins.js imports it; nothing that is checked imports tools.*. So npm run typecheck never type-checks them, despite their @ts-check header.
Related: #8708 (listed in #8712) made npm run typecheck pass; these two files were added later by #8694 and fall outside it.
Possible cause (speculation only)
A guess: the include pattern was written when all CLI entry points lived directly in cli/, and the new subdirectory files were assumed to be covered.
- Ngôn ngữ chính
- OCaml
- Star
- 7.5k
- Fork
- 484
- Merge trung bình
- 19 giờ 46 phút
- Pull request đã merge (30 ngày)
- 81
Chuẩn bị môi trường
Khởi chạy dev container của dự án ngay trên trình duyệt, bằng tài khoản GitHub của bạn.
- Không có Dockerfile hay tệp Docker Compose
- Không có mẫu pull request
- Đọc 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.
Issue khác của rescript-lang/rescript
-
Độ khó 3/5 Nửa ngày Mức phù hợp với người mới 58/100
rescript-lang/rescript#8757 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 40/100
rescript-lang/rescript#8753 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 22/100
rescript-lang/rescript#8733 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
rescript-lang/rescript#8727 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
rescript-lang/rescript#8726 ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của rescript-lang/rescript
Issue tương tự
-
PacReq: mangowcĐang mởpackage request
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
pacstall/pacstall-programs#9475 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Add `pdfcpu` to the pantryĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
Maintainer thường phản hồi trong vòng 1 ngày
-
[dsh-plugin.org | dsh-plugin-hub] plugin distribution incomplete: yjh051108/dsh-routing-suiteĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 71/100
yjh051108/dsh-routing-suite#227 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
JoviDeCroock/pracht#432 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
munich-quantum-toolkit/workflows#492 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày