openzim / openzim/python-scraperlib
[next major] Remove deprecated 'convert' parameter for optimize_image function inside optimization.py
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- Python
- Star
- 31
- Fork
- 27
- Merge trung bình
- 3 ngày 7 giờ
- Pull request đã merge (30 ngày)
- 2
Mô tả
Once #289 gets merged, the convert parameter will be deprecated and should be removed as the conversion happens automatically inside each optimize_xxx method. We no longer need to pass the dst format string via convert parameter. Users should use the parameter dst_format instead to pass the dst format.
Hướng dẫn đóng góp
Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này
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
Sau khi #289 được merge, bắt đầu tại optimize_image trong optimization.py và lần theo các caller cùng các test của nó. Thay thế việc sử dụng tham số đã deprecated convert bằng dst_format khi cần, sau đó xác minh rằng việc chuyển đổi vẫn được từng phương thức optimize_xxx xử lý và các test bị ảnh hưởng đều pass.
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
- performance
- Loại issue
- Tái cấu trúc
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- 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
- 68/100