TheAlgorithms / TheAlgorithms/Python

Should we add matchify to pre-commit

Đang mở
#15,337 8 bình luận 0 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

enhancement
Ngôn ngữ chính
Python
Star
225k
Fork
51.1k
Merge trung bình
1 ngày 23 giờ
Pull request đã merge (30 ngày)
156

Mô tả

Feature description

@priya-sundaram-dev, please evaluate if we should add https://github.com/15r10nk/matchify to our codebase.

Python's match statement is powerful and we should encourage contributors to use it where it makes sense.

On Python 14t, please run uv tool install matchify && matchify data_structures, which should modify six files.

Do these changes look correct and useful? Do you recommend we add this tool to pre-commit?

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. 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.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Hướng nghiên cứu

Chạy uv tool install matchify && matchify data_structures trên Python 14t theo yêu cầu, sau đó kiểm tra sáu tệp đã được sửa đổi. Đánh giá xem các phép biến đổi có chính xác và hữu ích đối với codebase này hay không, cũng như liệu matchify có nên thuộc pre-commit hay không. Hoàn thành nghĩa là ghi lại các phát hiện và đưa ra khuyến nghị rõ rà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ệ
python
Lĩnh vực
tooling
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
Sôi nổi
Độ rõ ràng
Khá rõ ràng
Mức phù hợp với người mới
45/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.