SAP / SAP/python-pyodata

fromisotime cleanup when 3.11 is Python miniumum

Abierto Apto para principiantes
#304 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

chore
Lenguaje dominante
Python
Estrellas
268
Forks
100
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

Matrix builds for #303 exposed change in fromisotime for fractional-seconds between Python 3.10 and 3.11. Since 3.10 EOL is quite soon, adding what can be changed when support for 3.10 will be dropped as well.

Cleanup tasks for when Python 3.11 becomes minimum version

  • parse_datetime_literal in pyodata/v2/model.py: remove fractional-seconds padding workaround (the if '.' in value block), fromisoformat accepts any number of fractional digits in 3.11+
  • EdmDateTimeOffsetTypTraits.from_literal in pyodata/v2/model.py: remove fractional-seconds padding workaround (the if '.' in normalized block) for the same reason
  • EdmDateTimeOffsetTypTraits.from_literal: remove the .replace('Z', '+00:00') normalisation, fromisoformat accepts Z natively in 3.11+

Guía de contribución

Abrir la guía de contribución

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.

Línea de trabajo

Abre pyodata/v2/model.py e inspecciona parse_datetime_literal y EdmDateTimeOffsetTypTraits.from_literal. Confirma el comportamiento de Python 3.11 con fromisoformat y, después, elimina los bloques de relleno de segundos fraccionarios y la sustitución de Z descritos en el issue. Se considera hecho cuando se hayan eliminado las soluciones alternativas de compatibilidad, mientras que los segundos fraccionarios y las entradas con Z sigan siendo compatibles.

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

Evaluación

Stack tecnológico
python
Área
api
Tipo de issue
Refactorización
Dificultad
2/5
Tiempo estimado
1-3 horas
Estado de actividad
Tranquilo
Claridad
Bien especificado
Aptitud para principiantes
78/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.