Remove dep on traitlet
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 35/100
Línea de trabajo
Comienza localizando dónde el paquete comm importa traitlets.utils.importstring y compáralo con la implementación de importstring.py de traitlets enlazada en la issue. Elimina la dependencia manteniendo la compatibilidad declarada del paquete con Python 3.6 y, después, ejecuta la suite de pruebas existente; la issue no especifica archivos ni pruebas concretos.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Hi!
as we discovered, the dependency on traitlet >= 5 means that actually, the comm project requires python >= 3.7 contrary to what is published (claims to support python 3.6)
I've seen that the traitlet dependency is used just in order to import this : https://github.com/ipython/traitlets/blob/main/traitlets/utils/importstring.py , which is pretty stable and easy
I suggest to just copy the method in comm package and remove the dep on traitlet all-together
what do you think ?
- Lenguaje dominante
- Python
- Estrellas
- 12
- Forks
- 19
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
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.
Issues similares
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
canonical/paas-charm#368 · 1 comentario ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
tech debt
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
addition to tracking list Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
StevenBlack/hosts#3256 ·
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
qualcomm/qai-appbuilder#275 ·