E131 applies inconsistently with nested calls
Đá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
- 35/100
Hướng nghiên cứu
Bắt đầu bằng cách tái hiện hai ví dụ phép gán với pycodestyle 2.6.0 và so sánh các kết quả E131 khi tên biến thay đổi. Theo dõi cách kiểm tra E131 xử lý các lệnh gọi phương thức lồng nhau và thụt lề treo; hoàn thành nghĩa là định dạng hợp lệ tương đương không còn thay đổi hành vi dựa trên độ dài của đích phép gán, với phạm vi kiểm thử hồi quy cho các ví dụ.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
I often use an indentation format like this for long method chains with something like SQLAlchemy, which seems to be acceptable according to my reading of PEP8 and doesn't raise any errors when pycodestyle is run:
rows = db.engine.execute(
table.select()
.where(table.c.column1 == 'something')
.where(table.c.column2.is_(None))
.order_by(table.c.column3)
.limit(5)
)
However, if the name of the variable being assigned to is a particular length, E131 - continuation line unaligned for hanging indent is raised:
aaaaaaaa = db.engine.execute(
table.select()
.where(table.c.column1 == 'something') # E131
.where(table.c.column2.is_(None)) # E131
.order_by(table.c.column3) # E131
.limit(5) # E131
)
It seems to be related to having the = operator aligned with the indented lines? Adding just a single character to the left side of the assignment will remove the E131.
aaaaaaaab = db.engine.execute(
table.select()
.where(table.c.column1 == 'something')
.where(table.c.column2.is_(None))
.order_by(table.c.column3)
.limit(5)
)
Is this an issue in pycodestyle, or do I just not understand what's going on here?
$ pycodestyle --version
2.6.0
- Ngôn ngữ chính
- Python
- Star
- 5.2k
- Fork
- 760
- 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ự
-
bot-triaged module: LrScheduler module: optimizer triaged
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
Maintainer thường phản hồi trong vòng 1 ngày
-
documentation glm5 llm-model
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
vllm-project/vllm-ascend#19062 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Author Page: shuai-zhao-llmĐang mởauthor-page correction
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
acl-org/acl-anthology#10496 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Issue Type: Bug Report
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
celery/celery#10875 · 1 bình luận ·
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 76/100
SasanLabs/VulnerableApp#836 ·
Maintainer thường phản hồi trong vòng 1 ngày