matrix-org / matrix-org/mscbot-python
Users (who aren't whitelisted) who post top-level to an proposal should have their comment automatically moved to a PR review comment
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- Python
- Star
- 7
- Fork
- 2
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Mô tả
The Spec Core Team often has the manual task of cleaning up comments that appear from contributors at the top-level of the MSC (a PR comment, rather than a PR review comment). Cleaning them up involves copying the contents of the comment, hiding it, then starting a new thread on the MSC (typically at line 1) and pasting the contents there as a quote. Along with some text to inform the user as to why it was necessary.
This is time-consuming manual work. And sometimes when we don't catch these comments quickly enough, other people will reply with another top-level comment, making the problem worse.
It would be nice if mscbot could take care of doing that automatically, along with a canned explanation. It should support a config option that excludes certain users (SCT, guardians, MSC author) who occasionally do need to post top-level from the correction.
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
Issue không nêu tên file, test hay entry point nào. Hãy bắt đầu bằng việc xác định code xử lý các bình luận pull request của GitHub và configuration, sau đó theo dõi cách các bình luận cấp cao nhất và việc loại trừ người dùng được biểu diễn. Được xem là hoàn tất khi các bình luận đủ điều kiện được chuyển vào các review thread kèm phần giải thích soạn sẵn, trong khi những người dùng bị loại trừ không bị ảnh hưởng.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- github, python
- Lĩnh vực
- devtools
- 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
- Đình trệ
- Độ rõ ràng
- Khá rõ ràng
- Mức phù hợp với người mới
- 35/100