Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix typing support #65

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fix typing support #65

wants to merge 1 commit into from

Conversation

rg2011
Copy link
Collaborator

@rg2011 rg2011 commented May 22, 2023

Al intentar usar la librería tc-etl-lib desde otros proyectos, siempre tenemos problemas con los tipos. mypy se queja de que la librería no está tipada.

Por lo que parece, para exportar información de tipo en los paquetes de instalación de las librerías, hay que seguir unos pasos que están en la PEP 0561 (https://peps.python.org/pep-0561/)

Esta PR implementa la primera de las opciones descrita en la PEP (fichero py.typed). Adicionalmente, corrige todos los warnings que genera mypy al validar el código de la propia librería.

@rg2011 rg2011 requested a review from fgalan May 22, 2023 13:16
@@ -0,0 +1,2 @@
# Marker file to support type hints in exported package
# See https://peps.python.org/pep-0561/
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

¿Para que sirve este fichero? Ahora mismo por lo que veo estaría vacio (ya que entiendo que las líneas que empiezan por # se interpretan como comentarios)

Copy link
Collaborator Author

@rg2011 rg2011 May 23, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Es un requisito de la PEP para exportar la información de tipado de la biblioteca: https://peps.python.org/pep-0561/#packaging-type-information . Por lo que entiendo de la PEP, el fichero es solo un marcador pero debe existir y distribuirse junto con la librería, incluyéndolo en el setup.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, aclarado. NTC.

@fgalan
Copy link
Contributor

fgalan commented May 23, 2023

Creo que estaría bien aprovechar esta PR para incluir una GitAction que verifique el Python typing en este repositorio. Sino, corremos el riesgo de romper y no darnos cuenta, con lo que podriamos generar versiones de tc-etl-lib que a ciertos usuarios (los que no tenga Python typing en sus estornos) les irian bien y a ciertos otros (los que sí lo tengan) mal.

Supongo que mirando como se está haciendo en otros repos (pe. el de urbo-deployer) está muy a tiro incluirlo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants