Normalize dot segments before matching maintainer paths
Maintainer thường phản hồi trong vòng 1 ngày
Chưa có ai nhận issue nà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
- 84/100
Hướng nghiên cứu
Bắt đầu với tools/get_maintainer.py và tái hiện hai lệnh trong issue để so sánh các đường dẫn canonical và các đường dẫn có phân đoạn dấu chấm. Bổ sung coverage bằng cách sử dụng các contact section và contact mặc định khác nhau, sau đó xác minh rằng các đường dẫn tương đương tương đối với repository chọn cùng những maintainer, trong khi các dấu phân cách của Windows và các đường dẫn có escape vẫn giữ nguyên hành vi hiện tại.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Problem
The maintainer lookup matches raw string prefixes after replacing backslashes. A common ./ prefix bypasses every relevant MAINTAINERS entry, routing a maintained file to the default. Equivalent paths should select the same ownership section.
Starting points: tools/get_maintainer.py.
Reproduction
Executed on Linux amd64 with Python 3.14.7. The affected files at local ade878134519b0f957ca28a23c6f9cd98e43dff3 are byte-identical to canonical master 286dad9c8fcb0616f7c719c72cbf453a7d9925fd.
python3 tools/get_maintainer.py front/parser/src/lib.rs
python3 tools/get_maintainer.py ./front/parser/src/lib.rs
The first command prints Maintainers to CC:; the second prints No specific maintainers found. Using default:. Current matching/default email addresses happen to be equal, but section selection is wrong and will misroute once contacts differ.
Scope
Easy: normalize repository-relative path spelling before matching while keeping ownership boundaries intact. This is a lookup behavior defect, separate from refreshing stale ownership entries in #352. Do not change maintainer contacts.
Completion criteria
- Canonical paths and equivalent ./ or internal dot-segment spellings select the same maintainers.
- Use a fixture with different section/default contacts to catch incorrect fallback.
- Preserve Windows separator handling and do not treat paths escaping the repository as owned files.
- Ngôn ngữ chính
- Rust
- Star
- 56
- Fork
- 21
- Merge trung bình
- 2 giờ 51 phút
- Pull request đã merge (30 ngày)
- 53
Chuẩn bị môi trường
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 wavefnd/Wave
-
bug good first issue needs testing
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
wavefnd/Wave#810 · 1 người được giao ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug good first issue needs testing
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
wavefnd/Wave#808 · 1 người được giao ·
Maintainer thường phản hồi trong vòng 1 ngày
-
enhancement good first issue needs testing
Độ 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
-
enhancement good first issue needs testing
Độ 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
-
enhancement good first issue needs testing
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
Maintainer thường phản hồi trong vòng 1 ngày
Issue tương tự
-
Độ 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
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
trezor/trezor-firmware#7997 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
smol-machines/smolvm#1489 · 1 bình luận · 1 reaction ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 88/100
Maintainer thường phản hồi trong vòng 1 ngày