Avoid use transform directly

Đang mở
#2 1 bình luận 0 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

Đánh giá

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

Hướng nghiên cứu

Bắt đầu bằng cách xác định việc sử dụng trực tiếp Node's Transform API trong module và xem xét package manifest để kiểm tra các quy ước về dependency. Thay thế việc sử dụng đó bằng through2, sau đó xác minh rằng việc phân tích JSON Lines vẫn hoạt động và package đã sẵn sàng cho bản phát hành v1.0.0 được đề xuất.

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

Mô tả

Hey,

I'm seeing you are using Transform directly from Node API.

Event this is currently working, could be don't work if the next Node version change the node version.

Then your module will be broken.

A better approach is use Transform as dependency using through2

In fact, it was created exactly for this reason, I recommend you read this author blog post:
https://r.va.gg/2014/06/why-i-dont-use-nodes-core-stream-module.html

PS: Could be a good opportunity to use it and ship v1.0.0 😄

Ngôn ngữ chính
JavaScript
Star
25
Fork
2
Không có pull request nào được merge trong 30 ngày

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

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. 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.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của LinusU/node-jsonlines

Tất cả issue của LinusU/node-jsonlines

Issue tương tự

Thêm issue về JavaScript

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.