Checking 4 spaces indent in C files in CI
Maintainer thường phản hồi trong vòng 1 ngày
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 2/5
- Thời gian dự kiến
- Nửa ngày
- Mức phù hợp với người mới
- 45/100
Hướng nghiên cứu
Xem xét thiết lập CI và các mã nguồn C trong ext/openssl/, sử dụng mẫu check_no_tab.sh được đề xuất làm điểm bắt đầu. Xác nhận rằng kiểm tra báo cáo các dòng chứa tab và vượt qua khi không có tab; được xem là hoàn tất khi kiểm tra chạy trong CI cho các tệp C.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Seeing the PR https://github.com/ruby/openssl/pull/973 (thanks for the PR!), I am considering if we can check if the indent is 4 spaces without tabs in CI. Right now I am seeing the following tools.
- clang-format: https://clang.llvm.org/docs/ClangFormat.html
- GNU indent: https://www.gnu.org/software/indent/
If we only check if C files don't include tabs. We may just apply the following kind of script.
$ cat check_no_tab.sh
#!/bin/bash
set -eu
if grep -rPn '\t' ext/openssl/; then
echo "FAIL"
exit 1
fi
echo "OK"
exit 0
OK case:
$ ./check_no_tab.sh
OK
Error case:
$ ./check_no_tab.sh
ext/openssl/ossl_ssl.c:3302: DefIVarID(hostname);
FAIL
What do you think?
- Ngôn ngữ chính
- C
- Star
- 276
- Fork
- 200
- Merge trung bình
- 15 giờ 35 phút
- Pull request đã merge (30 ngày)
- 7
Chuẩn bị môi trường
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 ruby/openssl
-
Độ 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
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 48/100
ruby/openssl#1082 · 4 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
ruby/openssl#1075 · 4 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Unchecked *_set_* callsĐang mở
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
ruby/openssl#1038 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Add a method OpenSSL::PKey#sizeĐang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 45/100
ruby/openssl#988 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Failed to resolve logo sourceĐang mởbug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
fastfetch-cli/fastfetch#2628 ·
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 78/100
FujiNetWIFI/fujinet-firmware#1736 ·
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 3 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 86/100