prometheus / prometheus/client_python
[Question] Is "REGISTRY" meant to be a decorator for "Collectors"?
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Python
- Estrellas
- 4.4k
- Forks
- 876
- Merge medio
- 8 d 4 h
- PR fusionados (30 d)
- 1
Descripción
Browsing through the code I've found a function called write_to_textfile in the exposition module. This function signature looks like this:
def write_to_textfile(path, registry):
"""Write metrics to the given path.
pass
If you read the implementation of the function, turns out the expectation is not really a registry but instead anything that implements (something like an interface) the .collect() method.
Is that by design?
If so, would it be welcomed the addition of type hints throughout the code? I would be happy to add them and create a PR.
Guía de contribución
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.
Línea de trabajo
Comienza en el módulo de exposición, en write_to_textfile, y compara su parámetro registry con el comportamiento de .collect() descrito en el issue. Antes de hacer cambios, determina si se desean anotaciones de tipos en todo el código base de Python; hacerlo requeriría acordar el alcance y contar con una decisión de los maintainers.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- python
- Área
- observability-sre
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Estado de actividad
- Estancado
- Claridad
- Necesita aclaración
- Aptitud para principiantes
- 25/100