Add `discover()` method to (all) finders
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- Python
- Star
- 77.2k
- Fork
- 35.9k
- Chỉ số merge pull request
- Chỉ số pull request đang chờ
Mô tả
Feature or enhancement
Proposal:
It would be nice to implement discover() for all finders where that can be reasonably achieved.
-
BuiltinImporter -
FrozenImporter -
WindowsRegistryFinder -
PathFinder -
FileFinder -
zipimporter
Has this already been discussed elsewhere?
No response given
Links to previous discussion of this feature:
No response
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.
Hướng nghiên cứu
Bắt đầu bằng cách đọc các triển khai discover() hiện có cho PathFinder và FileFinder, sau đó kiểm tra các kiểu finder còn lại: BuiltinImporter, FrozenImporter, WindowsRegistryFinder và zipimporter. Xác định finder nào có thể hỗ trợ phương thức này một cách hợp lý và xác minh rằng mỗi triển khai được chọn hoạt động nhất quán với các triển khai hiện có. Công việc được xem là hoàn tất khi các finder phù hợp chưa được kiểm tra đã được triển khai và được bao phủ bởi các bài kiểm thử liên quan của dự án.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- python
- Lĩnh vực
- backend
- Loại issue
- Tính năng
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức độ hoạt động
- Ít trao đổi
- Độ rõ ràng
- Khá rõ ràng
- Mức phù hợp với người mới
- 48/100