Enhancement: relax os.path.common input requirements
Đang mở
Chưa có ai nhận issue này.
stdlib
type-feature
- Ngôn ngữ chính
- Python
- Star
- 77.2k
- Fork
- 36k
- Chỉ số merge pull request
- Chỉ số pull request đang chờ
Mô tả
Feature or enhancement
os.path.commonpath is needlessly strict requiring its paths argument to be a Sequence. The underlying algorithm does not require it and the restriction is an implementation artifact.
Pitch
- Faster code: current implementation goes over its input while, in principle, one iteration is sufficient
- Relaxed argument requirement:
Sequenceis replaced withIterable
Previous discussion
https://discuss.python.org/t/os-path-commonpath-doesnt-need-a-sequence/21543/13
Linked PRs
- gh-99964
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 từ điểm vào os.path.commonpath và kiểm tra việc triển khai hiện có cùng các bài kiểm thử cho đối số paths của nó. So sánh hành vi với cuộc thảo luận được liên kết, sau đó xác minh rằng các đầu vào iterable được chấp nhận, việc xử lý được thực hiện trong một lượt và hành vi hiện có của path vẫn được giữ nguyê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
- operating-systems
- Loại issue
- Tính năng
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức phù hợp với người mới
- 25/100