[next-major] Remove traces of CamelCase
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 35/100
Línea de trabajo
Comienza por las declaraciones del wrapper de Python mostradas en el issue y localiza los nombres CamelCase restantes en las APIs de writer, suggestion y search. Renombra esos métodos y parámetros expuestos a snake_case, elimina las excepciones de nomenclatura asociadas y verifica que las referencias y las comprobaciones utilicen la nueva API antes de la próxima versión major.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
As libzim uses CamelCase, we sometimes followed the convention to stick to the pure-wrapper way.
Given case is not that important and we don't follow it everywhere (it's become minor overtime), we should just remove the remaining traces and use snake case everywhere.
It's a tiny change but obviously breaks the API so should not be merged until approaching next major release
# writer
def config_nbworkers(self, nbWorkers: int) -> Self: ... # noqa: N803
def set_mainpath(self, mainPath: str) -> Self: ... # noqa: N803
def add_redirection(
self,
path: str,
title: str,
targetPath: str, # noqa: N803
hints: dict[Hint, int],
) -> None: ...
def add_alias(
self,
path: str,
title: str,
targetPath: str, # noqa: N803
hints: dict[Hint, int],
) -> None: ...
# suggestion
def getEstimatedMatches(self) -> int: ... # noqa: N802
# search
def getEstimatedMatches(self) -> int: ... # noqa: N802
def getResults(self, start: int, count: int) -> SearchResultSet: ... # noqa: N802
- Lenguaje dominante
- Python
- Estrellas
- 109
- Forks
- 29
- Merge medio
- 9 d 1 h
- PR fusionados (30 d)
- 1
Preparar el entorno
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de openzim/python-libzim
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
openzim/python-libzim#268 · 4 comentarios · 1 reacción ·
-
Release 3.14.0Posiblemente ocupada @benoit74 la tomó hace 21 días. Abiertotask
openzim/python-libzim#267 · 1 asignado ·
-
Code review: writer-path use-after-free, sys.modules corruption, GIL held across all reader I/OAbierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
openzim/python-libzim#266 ·
-
Build for ABI3Posiblemente ocupada @rgaudin la tomó hace 46 días. Abiertoenhancement
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
openzim/python-libzim#264 · 1 comentario ·
-
enhancement upstream
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
openzim/python-libzim#241 ·
Todos los issues de openzim/python-libzim
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-2 días Aptitud para principiantes 70/100
-
FingerprintSplitter raises ZeroDivisionError when int(frac_train * len(dataset)) floors to zeroAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
Los mantenedores suelen responder en 7 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
lmstudio-ai/mlx-engine#376 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
pyiron/bagofholding#166 ·