Review "failing" sample files for multiple failures, split into separate files.
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức phù hợp với người mới
- 35/100
Hướng nghiên cứu
Xem xét các tệp trong snifftests/files có các dòng đầu tiên liệt kê nhiều quy tắc bị lỗi. Kiểm tra xem từng lỗi có riêng biệt hay không; tách các tệp khi có thể, hoặc xem xét phần chồng lấn giữa các bộ quy tắc CakePHP và PSR-2 khi các lỗi bao phủ cùng một vấn đề. Hoàn thành khi các mẫu bị ảnh hưởng có các lỗi nguyên tử hoặc một bộ quy tắc đã được điều chỉnh.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Some of the sample files currently fail multiple sniffs. Our testing harness can catch multiple sniff failures per file, but it makes the tests relatively inflexible since the files aren't "atomic" compared to the sniff rules.
It's also possible for the test files to be failing tests they really shouldn't be, such as overlapping rules that cover the same "problem" with the code. In these cases, the ruleset needs to be modified to try to only include one relevant rule to cover the failure.
Bottom line is: The files in snifftests/files that contain first lines like this:
<?php //~Standard.Section.Sniff.Rule, Another.Failing.Rule.Name
(The ones that define multiple failing rules.) Those need to be reviewed and split up where possible, or the ruleset modified to not cause overlapping failures. (The CakePHP sniffs are responsible for a lot of overlap against PSR-2, and typically the Cake sniffs lack the "autofix" capability the PSR-2 sniffs include.)
- Ngôn ngữ chính
- PHP
- 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 loadsys/loadsys_codesniffer
-
enhancement
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 35/100
loadsys/loadsys_codesniffer#51 · 1 bình luận ·
-
question
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 20/100
-
.editorconfig to prevent stripping trailing space, adding final newline in snifftests/files/.Đang mởenhancement help-wanted
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 45/100
-
enhancement help-wanted
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
-
bug help-wanted
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
Tất cả issue của loadsys/loadsys_codesniffer
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
scanaislop/aislop#482 ·
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 62/100
Maintainer thường phản hồi trong vòng 1 ngày
-
documentation
Độ khó 2/5 1-3 giờ 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 82/100
humanmade/publication-checklist#73 · 1 bình luận ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
Werkstattl/OpenBlogware#92 ·