Extraneous space inserted after trailing comma
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
- 45/100
Hướng nghiên cứu
Bắt đầu bằng cách xác định các điểm vào của formatter cho các cấu trúc enum, set-like, record và vector, sau đó kiểm tra cách các dấu phẩy ở cuối được hiển thị. Tái hiện các ví dụ từ issue và so sánh từng đầu ra với hành vi của enum. Hoàn tất khi các dấu phẩy ở cuối được giữ nguyên mà không có khoảng trắng thừa khi không có đối số tiếp theo, với độ bao phủ cho các cấu trúc được liệt kê.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
In many places Zeek allows trailing commas. This seems to be handled inconsistently.
For enums I see expected behavior:
type E: enum { a, b, c, };
For set-like constructs I see an additional space:
const s = { 1, 2, 3, };
# ^
const r: RecordType = [ $a=1, $b=2, $c=3, ];
# ^
We probably have the same issue for function-like constructors, e.g.,
const a = vector(1, 2, 3, );
# ^
The issue seems to be that we insert an space after , in the contexts even though it is not followed by more arguments.
- 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 48/100
zeek/zeekscript#91 ·
-
Độ 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 ·
Tất cả issue của zeek/zeekscript
Issue tương tự
-
fix formatting of the sample database page of the docsCó thể đã có người làm @Aaqibhafeezkhan đã nhận hôm nay. Đang mởdocumentation good first issue help wanted
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 72/100
chaoss/CollectOSS#544 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
area/plugin good first issue
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 85/100
kestra-io/plugin-soda#137 ·
-
Độ khó 2/5 Nửa ngày Mức phù hợp với người mới 62/100
ibm-granite/granite.build#504 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
`LazyFrameRemote` and `ExecuteRemote` sinks do not accept `Path`, like `LazyFrame` sinks do.Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
Maintainer thường phản hồi trong vòng 1 ngày