[QUESTION] Is it correct to "throw" on invalid name and/or invalid version?
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 5/5
- Thời gian dự kiến
- Hơn một tuần
- Mức phù hợp với người mới
- 25/100
- Loại issue
- Tính năng
- Độ rõ ràng
- Cần làm rõ
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- javascript
- Lĩnh vực
- tooling
Hướng nghiên cứu
Start by reviewing the normalization function and the validName and validVersion checks shown in the issue. Determine the intended behavior for invalid metadata, including whether processing should stop and whether in-place packageData may remain partially normalized; done requires a documented decision and corresponding behavior.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
In original async code, callback was immediately called with an error object
var ret = validName(file, data)
if (ret !== true) return cb(ret);
ret = validVersion(file, data)
if (ret !== true) return cb(ret);
I wonder if this is desirable if this function would be used inside npm-www for example. If requirements for name or versions get more strict (I admit, maybe far-fetched) the whole normalization process stops. Considering the packageData is modified in-place, the data after the throw is then unnormalized (or half-normalized, depending on where the throw happened).
- Ngôn ngữ chính
- JavaScript
- Star
- 204
- Fork
- 53
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Chuẩn bị môi trường
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 npm/normalize-package-data
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 45/100
npm/normalize-package-data#105 ·
-
feature
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 32/100
-
bug
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 25/100
-
feature
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 35/100
npm/normalize-package-data#83 · 1 bình luận ·
-
bug
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 35/100
npm/normalize-package-data#75 · 2 reaction ·
Tất cả issue của npm/normalize-package-data
Issue tương tự
-
refactor
Độ khó 2/5 Nửa ngày Mức phù hợp với người mới 84/100
Maintainer thường phản hồi trong vòng 5 ngày
-
translation
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
ciderapp/translations#87 · 1 bình luận ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
Maintainer thường phản hồi trong vòng 1 ngày
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 67/100
Maintainer thường phản hồi trong vòng 1 ngày
-
component: split-view platform: windows
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 74/100
zen-browser/desktop#15616 · 1 reaction ·
Maintainer thường phản hồi trong vòng 1 ngày