Wrong TypeScript definitions

Đang mở
#4 0 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
Lỗi
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Đình trệ
Công nghệ
typescript
Lĩnh vực
tooling

Hướng nghiên cứu

Bắt đầu với DefinitelyTyped PR 53304 và các định nghĩa @types/jsonlines, sau đó so sánh giá trị được sự kiện data phát ra với phần triển khai của dự án này. Được coi là hoàn tất khi khai báo TypeScript bên ngoài biểu diễn chính xác mọi giá trị JSON hợp lệ được data phát ra; issue này không xác định tệp hoặc kiểm thử nào trong repository này.

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

Mô tả

I know TypeScript definitions aren't part of this package, but I thought I'd bring attention to the fact that someone wrote TypeScript definitions for this package in @types/jsonlines (here), and they're not accurate.

Specifically, the data event was defined as emitting an interface called Lines, which is defined as:

export interface Line {
    data: string;
    type: string;
}

Which is pretty strange to me, because I've read the code in this project, and there is no such data structure ever defined. If I'm correct, the data event just emits any valid JSON value.

Mentioning @raineorshine, as they are the author of those TypeScript Definitions.

Ngôn ngữ chính
JavaScript
Star
25
Fork
2
Chỉ số merge pull request
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.