Mismo valor en constantes

PabloCastellano / bormeparser

A Python library for parsing BORME files (Boletín Oficial del Registro Mercantil in Spain).

GNU General Public License v3.0

46 stars 20 forks source link

Mismo valor en constantes #111

Closed fquinto closed 1 year ago

fquinto commented 1 year ago

Hola, ¿es un error de tipo el contener dos constantes de mismo valor 32?

CIERRE_PROVISIONAL_REVOCACION_NIF = 32
REACTIVACION_DE_LA_SOCIEDAD = 32

https://github.com/PabloCastellano/bormeparser/blob/efb534ed0697df2486f0cb1ea92aac3b08437827/bormeparser/acto.py#L52

¿por qué los "huecos"? Gracias

PabloCastellano commented 1 year ago

Hola! Aunque no afecte al parser, efectivamente sí que parece un error y debería ser REACTIVACION_DE_LA_SOCIEDAD = 33.

Como se puede ver, este proyecto está desmantenido y no se han ido implementando las nuevas funcionalidades de Python. Por si quieres hacer una PR, creo que lo mejor sería convertir la clase ACTO a Enum (que fue añadido en Python 3.4).

fquinto commented 1 year ago

Hola, gracias por contestar, no tenía claro la afectación... Acabo de hacer un PR (https://github.com/PabloCastellano/bormeparser/pull/112) con varias mejoras y un nuevo flag para forzar descargas. Intentaré implementar alguna mejora más como la que has indicado.