openzim / openzim/python-scraperlib
[next major] Remove deprecated 'convert' parameter for optimize_image function inside optimization.py
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Python
- Sterne
- 31
- Forks
- 27
- Ø Merge
- 3 T. 7 Std.
- Gemergte PRs (30 T.)
- 2
Beschreibung
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.
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Nachdem #289 zusammengeführt wurde, beginne in optimization.py bei optimize_image und verfolge dessen Aufrufer und Tests. Ersetze die Verwendungen des veralteten Parameters convert bei Bedarf durch dst_format und überprüfe anschließend, dass die Konvertierung weiterhin von jeder optimize_xxx-Methode behandelt wird und dass die betroffenen Tests bestehen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- python
- Bereich
- performance
- Issue-Typ
- Refactoring
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Ruhig
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 68/100