Contextual diff lines checked
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
Tái hiện ví dụ bằng cách pipe git diff HEAD~1 vào pycodestyle --diff, sau đó theo dõi cách xử lý đầu vào --diff trong tệp Python duy nhất của repository. Được xem là hoàn tất khi các dòng ngữ cảnh như example.py:2 bị bỏ qua, trong khi các dòng được thêm vẫn tạo ra những lỗi như mong đợi.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
pycodestyle --diff is great! More programs should accept diffs as input.
Unfortunately it appears to consider lines in the diff that are merely context. Consider the following diff:
diff --git a/example.py b/example.py
index 955628d..b6c2b66 100644
--- a/example.py
+++ b/example.py
@@ -1,2 +1,5 @@
def invalid_but_irrelevant():
print("I am invalid but also irrelevant")
+
+def invalid_but_relevant():
+ print("I am invalid and relevant")
Line 2 (print("I am invalid but also irrelevant")) was not added in this diff, but pycodestyle still finds the problem with its indentation:
user@host:~/gists/98f18739ab30fb69d6154364dc1eee7a$ git diff HEAD~1 | pycodestyle --diff
./example.py:2:3: E111 indentation is not a multiple of four
./example.py:4:1: E302 expected 2 blank lines, found 1
./example.py:5:3: E111 indentation is not a multiple of four
I expect --diff to only consider lines added by a diff; otherwise code that's problematic but outside the scope of a change causes that change to fail linting checks.
- Ngôn ngữ chính
- Python
- Star
- 5.2k
- Fork
- 756
- Merge trung bình
- 5 giờ 56 phút
- Pull request đã merge (30 ngày)
- 3
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 PyCQA/pycodestyle
-
DOC: E252 is not documentedĐang mở
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 82/100
PyCQA/pycodestyle#1294 · 1 bình luận ·
-
Typing and formattingĐang mở
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 25/100
PyCQA/pycodestyle#1302 · 3 bình luận ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 42/100
PyCQA/pycodestyle#1242 · 2 bình luận ·
-
Spurious E231 for nested format substitutionsCó thể làm lại được Pull request cho issue này đã bị đóng mà không được merge. Đang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 68/100
PyCQA/pycodestyle#1241 · 2 bình luận · 4 reaction ·
-
`exclude` is applied to `.`Đang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 45/100
PyCQA/pycodestyle#1217 ·
Tất cả issue của PyCQA/pycodestyle
Issue tương tự
-
SSM GetParameter throttling during concurrent workflow launchesCó thể đã có người làm @alexiswl đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 80/100
-
[Bug] Dev image build fails: `npm install` in the root Dockerfile runs out of memory (exit 134)Có thể đã có người làm @andest01 đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
Maintainer thường phản hồi trong vòng 1 ngày
-
enhancement good first issue
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 79/100
pesu-dev/discord_bot#97 ·
-
Độ khó 1/5 1-3 giờ Mức phù hợp với người mới 88/100
gradio-app/gradio#13964 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 93/100
BSData/horus-heresy-3rd-edition#3232 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày