Empty line before comment should (sometimes) be preserved
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức phù hợp với người mới
- 48/100
Hướng nghiên cứu
Bắt đầu với cách zeek-format xử lý các dòng trống trước các chú thích nhỏ, bằng cách so sánh các chú thích sau một sự kiện đã hoàn tất với các chú thích là nút đầu tiên sau {. Xác nhận rằng việc định dạng ví dụ giữ lại dòng trống và việc định dạng lại kết quả không làm thay đổi kết quả, trong khi các dòng trống ngay sau { vẫn bị xóa.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Currently zeek-format removes empty lines before any minor comments, e.g.,
event new_event(name: string, params: call_argument_vector)
{
print name;
}
# Should preserve empty line before.
first gets formatted to
event new_event(name: string, params: call_argument_vector)
{
print name;
}
# Should preserve empty line.
and then again (breaking idempotency) to
event new_event(name: string, params: call_argument_vector)
{
print name;
}
# Should preserve empty line.
I would have expect this to not change formatting at all.
zeek-format should still remove empty after e.g., { (comment first node in block).
Labelling as bug due to idempotency breaking.
- Ngôn ngữ chính
- Python
- Star
- 11
- Fork
- 4
- Merge trung bình
- 1 ngày 20 giờ
- Pull request đã merge (30 ngày)
- 1
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 zeek/zeekscript
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 45/100
zeek/zeekscript#137 ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 38/100
zeek/zeekscript#120 ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 45/100
zeek/zeekscript#83 · 1 bình luận ·
-
Function calls incorrectly formattedCó thể làm lại được Pull request cho issue này đã bị đóng mà không được merge. Đang mở
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
zeek/zeekscript#69 ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 45/100
zeek/zeekscript#63 ·
Tất cả issue của zeek/zeekscript
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
huggingface/sentence-transformers#4186 ·
Maintainer thường phản hồi trong vòng 5 ngày
-
documentation good first issue
Độ khó 2/5 Nửa ngày Mức phù hợp với người mới 72/100
sujithputta02/Esapay#13 ·
-
bot-triaged module: correctness (silent) module: mps module: NaNs and Infs triage review
Độ khó 2/5 Nửa ngày Mức phù hợp với người mới 62/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ 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 1 ngày
-
README: CI link/badge checkĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
SCIInstitute/SCIRun#2842 ·
Maintainer thường phản hồi trong vòng 3 ngày