Plugin registry compares plugins by equality, not identity
Maintainer thường phản hồi trong vòng 1 ngày
@00200200 đang làm issue này rồi.
Từ ngày 11/10/2026.
Đá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
- 62/100
Hướng nghiên cứu
Bắt đầu từ register(), get_name() và is_registered() trong trình quản lý plugin, các hàm này so sánh plugin bằng == hoặc in trên các giá trị của _name2plugin, và so sánh chúng với get_hookcallers(), hàm này đã dùng is. Tái hiện bằng ví dụ hai instance dataclass trong issue, rồi chuyển các phép so sánh sang so sánh định danh. Hoàn thành khi hai đối tượng plugin bằng nhau nhưng khác nhau có thể được đăng ký dưới các tên khác nhau và bộ kiểm thử hiện có vẫn chạy thành công.
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.
register() (plugin in self._name2plugin.values()), get_name() and is_registered() use ==, while get_hookcallers() uses is. Two distinct plugins that compare equal are treated as one:
@dataclass
class P:
@hookimpl
def myhook(self): ...
pm.register(P(), "one")
pm.register(P(), "two") # ValueError: Plugin already registered under a different name
Proposal: use identity everywhere. Small behavior change for plugins with a custom __eq__.
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
-
Defaulted positional-only hook parameters are forwarded as keywordsCó thể đã có người làm @adenzhou1350 đã nhận 2 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 · 1 bình luận ·
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 4 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
-
unregister() argument validation relies on assertCó thể đã có người làm @ekanshul đã nhận 3 ngày trước. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 42/100
pytest-dev/pluggy#762 ·
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 13 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 difficulty:2 good first issue help wanted
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
MakazhanAlpamys/Soup#1865 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
area: agents area: docs good first issue
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
openapi/gknowledge#23 ·
-
useless-suppression reported for deprecated-* messages that should be exemptCó thể đã có người làm @adityasinha-ghub đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
pylint-dev/pylint#11656 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
area:provisioning effort:S priority:P2 provider:local-libvirt risk:daytime-only status:in-progress type:bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
randomparity/kdive#3203 · 2 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
get() for StrEnumCó thể đã có người làm @FeJo5952 đã nhận hôm nay. Đang mởgood first issue
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
collective/icalendar#1921 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày