unregister() argument validation relies on assert
Maintainer thường phản hồi trong vòng 1 ngày
Đánh giá
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức phù hợp với người mới
- 42/100
Hướng nghiên cứu
The checks live in the unregister() method, which the issue names but does not locate in a file, so search the repository for it. Confirm that assert is what enforces the two conditions, then run the existing unregister tests to see what currently passes. Done means both error cases raise the proposed exceptions, but the maintainers have not yet agreed to the AssertionError breaking change, so check for a decision before writing code.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
🤖 Written by Claude Opus 5.5 via Claude Code for the pluggy maintainers; I prompted it, it did the work, I read it.
unregister() checks "one of name or plugin" and "plugin is not registered" with assert. Under python -O, pm.unregister(unregistered_plugin) silently returns the plugin, and pm.unregister() returns None.
Proposal: raise TypeError when neither argument is given, and ValueError when the plugin is not registered. Code that currently catches AssertionError would need to change.
Found while working on #759.
- Ngôn ngữ chính
- Python
- Star
- 1.7k
- Fork
- 172
- Merge trung bình
- 2 ngày 18 giờ
- Pull request đã merge (30 ngày)
- 21
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 pytest-dev/pluggy
-
Plugin registry compares plugins by equality, not identityCó thể đã có người làm Có pull request liên kết đang mở hoặc đã được merge. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
pytest-dev/pluggy#760 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Defaulted positional-only hook parameters are forwarded as keywordsCó thể đã có người làm @adenzhou1350 đã nhận 1 ngày trước. Đang mở
Độ khó 4/5 1-2 ngày Mức phù hợp với người mới 45/100
pytest-dev/pluggy#774 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
list_plugin_distinfo() keeps unregistered pluginsCó thể đã có người làm @RonnyPfannschmidt đã nhận 2 ngày trước. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 35/100
pytest-dev/pluggy#763 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Failures during add_hookspecs / historic replay leave partial stateCó thể đã có người làm Có pull request liên kết đang mở hoặc đã được merge. Đang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 50/100
pytest-dev/pluggy#761 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
[Downstream] devpi broken - trying to register `None`Có thể đã có người làm @bluetech đã nhận 11 ngày trước. Đang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 58/100
pytest-dev/pluggy#749 · 4 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của pytest-dev/pluggy
Issue tương tự
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
lugassawan/swe-workbench#799 ·
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 74/100
vllm-project/vllm#61131 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Non-string form field names raise AttributeError when files are attachedCó thể đã có người làm @nightcityblade đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
kubernetes-client/python#2735 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
desktop help wanted test
Độ 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