ASYNC103 for sync functions
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
- 62/100
Hướng nghiên cứu
Bắt đầu bằng cách tái hiện ASYNC103 đối với lệnh gọi đồng bộ được hiển thị trong src/trio/_core/_instrumentation.py và kiểm tra cách quy tắc phân biệt các hàm đồng bộ và bất đồng bộ. Hoàn tất khi kết quả dương tính giả không còn được báo cáo, trong khi các trường hợp ASYNC103 dự kiến vẫn được báo cáo; thêm hoặc cập nhật bài kiểm thử hồi quy liên quan sau khi xác định được vị trí của nó.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
I don't think sync functions can raise trio.Cancelled, but this is causing ASYNC103:
def call(
self,
hookname: str,
*args: object,
) -> None:
# snip
for instrument in list(self.data[hookname]):
try:
getattr(instrument, hookname)(*args)
except BaseException: # E
self.remove_instrument(instrument)
INSTRUMENT_LOGGER.exception(
"Exception raised when calling %r on instrument %r. "
"Instrument has been disabled.",
hookname,
instrument,
)
Error message:
src/trio/_core/_instrumentation.py:110:20: ASYNC103 BaseException block with a code path that doesn't re-raise the error. Consider adding an `except trio.Cancelled: raise` before this exception handler.
- Ngôn ngữ chính
- Python
- Star
- 26
- Fork
- 10
- 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
- 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 python-trio/flake8-async
-
Rename ASYNC126 to ASYNC4xxĐang mởrule improvement
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
python-trio/flake8-async#457 · 1 reaction ·
-
Extend ASYNC401 to other formsCó thể đã có người làm @anshurajbisoyi98-ctrl đã nhận 5 ngày trước. Đang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 68/100
python-trio/flake8-async#475 · 1 bình luận · 1 reaction ·
-
Warn about unused `noqa`sĐang mởnew rule
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 55/100
python-trio/flake8-async#454 · 1 bình luận ·
-
New rule: `ASYNC233` for blocking `pathlib` methodsCó thể đã có người làm @paranoa233 đã nhận 93 ngày trước. Đang mởgood first issue
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 15/100
python-trio/flake8-async#396 · 1 bình luận ·
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 25/100
python-trio/flake8-async#395 · 8 bình luận ·
Tất cả issue của python-trio/flake8-async
Issue tương tự
-
Độ khó 1/5 1-3 giờ Mức phù hợp với người mới 85/100
pytest-dev/pluggy#757 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 1/5 1-3 giờ Mức phù hợp với người mới 85/100
NousResearch/hermes-agent#134960 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
HTML backend: `<br>` leaks the internal sentinel U+E000 into list items, headings and captionsCó thể đã có người làm @morten-lagabote đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 67/100
docling-project/docling#4671 ·
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 70/100
Maintainer thường phản hồi trong vòng 1 ngày
-
good first issue hacktoberfest infra
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
Maintainer thường phản hồi trong vòng 1 ngày