Readability 📖
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
- 48/100
- Loại issue
- Tái cấu trúc
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- javascript
- Lĩnh vực
- api
Hướng nghiên cứu
Bắt đầu trong data.js ở các dòng 40-56 và theo dõi các lần gọi callback về sau trong hàm fetch để hiểu luồng bất đồng bộ. Đổi tên các biến callback mơ hồ theo cách mô tả rõ ràng mà không thay đổi hành vi, sau đó xác nhận rằng các bài kiểm thử API hiện có vẫn vượt qua và luồng fetch vẫn dễ đọc.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
https://github.com/fac-14/NDJS-API/blob/df236cfbc24fd346a72ea94df533a134282921c7/data.js#L40-L56
I know it is awkward chaining asynchronous callbacks - this will be much easier to sort out when you start using promises etc. - but this first line and the associated calls later in the fetch function lead to very difficult to interpret code. It is not the convention, but you might have made what is happening here a lot clearer with some more descriptive naming - instead of 'callback' and 'callback2', for example.
- Ngôn ngữ chính
- JavaScript
- Star
- 0
- Fork
- 0
- 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
Dự án này không cung cấp dev container, Dockerfile hay hướng dẫn đóng góp, nên bạn cần tự thiết lập môi trường: hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.
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 fac-14/NDJS-API
-
README needs contentĐang mở
Độ khó 1/5 1-3 giờ Mức phù hợp với người mới 82/100
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 35/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 35/100
-
Duplicated codeĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 42/100
-
🚨Use of innerHTMLĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 35/100
Tất cả issue của fac-14/NDJS-API
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 83/100
Maintainer thường phản hồi trong vòng 2 ngày
-
feedback simulation workshop
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
githubnext/gh-aw-workshop#4755 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
fix
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 72/100
idean3885/claude-ops-agent#728 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
deck: add to staging level: easy p-feature: Privacy Policy priority: MUST HAVE ready for product role: front-end size: 0.25pt Waiting on Stakeholder
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 61/100
hackforla/tdm-calculator#3654 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Add: Sony Vizha HDĐang mởcheck:failed feeds:add
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 78/100