1602 / 1602/compound

Consistent TypeOf

Đang mở Phù hợp với người mới
#627 2 bình luận 0 reaction 0 người được giao Xem trên GitHub
Ngôn ngữ chính
JavaScript
Star
1.6k
Fork
179
Chỉ số merge pull request
Không có pull request nào được merge trong 30 ngày

Mô tả

The code has mixed style between `TypeOf something === function` and `function === TypeOf something`.

https://github.com/1602/compound/blob/master/lib/controller-extensions.js#L137
https://github.com/1602/compound/blob/master/lib/controller-extensions.js#L304
https://github.com/1602/compound/blob/master/lib/controller-extensions.js#L307
https://github.com/1602/compound/blob/master/lib/compound.js#L167
https://github.com/1602/compound/blob/master/lib/compound.js#L257
https://github.com/1602/compound/blob/master/lib/compound.js#L264

Most are `if (typeof VAR === 'function') {` style. So it wouldn't hurt to stick to it.

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

Hướng nghiên cứu

Issue chỉ ra các dòng cụ thể trong lib/controller-extensions.js và lib/compound.js. Mở các tệp đó, tìm các kiểm tra typeof và thay đổi bất kỳ mẫu `function === typeof VAR` nào thành `typeof VAR === 'function'`. Chạy các bài kiểm tra hiện có để đảm bảo các thay đổi không phá vỡ chức năng.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Đánh giá

Công nghệ
node.js
Lĩnh vực
backend, tooling
Loại issue
Tái cấu trúc
Độ khó
1/5
Thời gian dự kiến
Dưới một giờ
Mức độ hoạt động
Đình trệ
Độ rõ ràng
Đặc tả rõ ràng
Mức phù hợp với người mới
75/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.