Would be nice if there was an iterable interface to the NodeWalker
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ính năng
- Độ 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 từ phần triển khai NodeWalker được trả về bởi .walker() và kiểm tra hành vi hiện có của next(). Thêm hỗ trợ iterable để for...of tạo ra cùng các sự kiện của walker và kết thúc khi quá trình duyệt hoàn tất; xác minh hành vi bằng các bài kiểm thử hiện có của dự án, mặc dù issue không nêu tên tệp kiểm thử nào.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Right now the NodeWalker instance returned by .walker() is not an actual javascript iterable.
Would be great to be able to do:
const walker = ast.walker()
for(event of walker) {
// stuff here
}
But right now that doesn't work.
It should be pretty simple to add, the following works when I already have a walker instance so just adding that before it is returned will work...
walker[Symbol.iterator] = function() {
return {
next() {
const data = walker.next();
if (data) {
return {
value: data
};
}
return {done: true};
}
};
};
- Ngôn ngữ chính
- JavaScript
- Star
- 1.6k
- Fork
- 233
- Merge trung bình
- 1 ngày 27 phút
- Pull request đã merge (30 ngày)
- 1
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Không có mẫu pull request
- Đọc hướng dẫn đóng góp
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 commonmark/commonmark.js
-
White space handling looks inconsistent between HTML block element vs HTML inline elementCó thể đã có người làm @mamicicekel đã nhận 11 ngày trước. Đang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 35/100
commonmark/commonmark.js#303 · 2 bình luận ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
commonmark/commonmark.js#287 · 1 reaction ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 20/100
commonmark/commonmark.js#275 ·
-
Delegate node walking to othersĐang mở
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 25/100
commonmark/commonmark.js#267 · 3 bình luận ·
-
Inconsistent escape of image altĐang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 42/100
commonmark/commonmark.js#264 · 1 reaction ·
Tất cả issue của commonmark/commonmark.js
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
Maintainer thường phản hồi trong vòng 2 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
bigskysoftware/htmx#4147 ·
Maintainer thường phản hồi trong vòng 5 ngày
-
Please add PNG images as wellĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
vshymanskyy/StandWithUkraine#316 ·
-
README: the green and red points are swapped (green = .fset tracking, red = .fset3 detection)Đang mở
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 90/100
-
bug/1-unconfirmed kind/bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
Maintainer thường phản hồi trong vòng 1 ngày