rfc3986 uses its own deprecated method - warning spam

Abierto
#123 2 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
35/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Estancado
Stack tecnológico
python
Área
backend

Línea de trabajo

Empieza en la implementación de URIMixin.resolve_with() e inspecciona cómo llama a URIReference.is_valid(). Sustituye la ruta de validación obsoleta por un Validator, ejecuta después las comprobaciones relevantes del proyecto y confirma que el exceso de advertencias ha desaparecido.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

In Version 2.0, in URIMixin.resolve_with() rfc3986 uses the deprecated URIReference.is_valid() instead of a Validator.

Apart from it being a bad look, it causes literally thousands of warnings in our pipeline (since resolve_with() is used often from the json-schema and jschon packages) - see below. This spams our logs and can make it hard to debug.

I made a fix: https://github.com/jpmaterial/rfc3986/tree/release/2.0 (and verified it), but there is no branch that offers itself for a pull request.

Are you interested in the fix (and releasing a 2.0.1 I guess) and if yes, on which branch should I request a pull?

image

Lenguaje dominante
Python
Estrellas
189
Forks
38
Merge medio
13 h 36 min
PR fusionados (30 d)
1

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de python-hyper/rfc3986

Todos los issues de python-hyper/rfc3986

Issues similares

Más issues de Python

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.