Skip to content

Cambio de version de pandas para python 3.13#121

Merged
fgalan merged 5 commits into
telefonicasc:masterfrom
jazminrillo-mb3:cambio_version_pandas
Apr 17, 2026
Merged

Cambio de version de pandas para python 3.13#121
fgalan merged 5 commits into
telefonicasc:masterfrom
jazminrillo-mb3:cambio_version_pandas

Conversation

@jazminrillo-mb3
Copy link
Copy Markdown
Contributor

No description provided.

Comment thread python-lib/tc_etl_lib/setup.py Outdated
'urllib3==1.26.16',
'psycopg2-binary>=2.9.5',
'pandas==2.2.2',
'pandas>=2.2.3',
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
'pandas>=2.2.3',
'pandas==2.2.3',

Por lo general, solemos preferir dependencias atadas (son más controlables y dan menos problemas, dejarlo como >= es en cierto modo una foto móvil)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Entendido. Cambiado a pandas == 2.2.3 en lugar de >=

Comment thread python-lib/tc_etl_lib/README.md Outdated
Comment on lines +577 to +578
0.18.0 (April 16th, 2026)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
0.18.0 (April 16th, 2026)

La marca de versión se incluye en PR de bump a parte.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Vale, la he quitado y la incluiré en el PR de bump luego

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Ok al eliminado de marca, pero la línea que describe el cambio hecho en la PR si habría que conservarla:

- Fix: change pandas==2.2.2 to pandas==2.2.3 in package requirements (Python 3.13 case)

(Perdona si no me expliqué con claridad)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Vale. Corregido. Gracias por la aclaración!

Copy link
Copy Markdown
Contributor

@fgalan fgalan left a comment

Choose a reason for hiding this comment

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

LGTM

@jazminrillo-mb3 ¿haces tú tambien la PR de bump de la 0.18.0, pls? Mirando como se hizo la del 0.17.0 (ya mezclada) entiendo te sería fácil hacer una análoga.

@fgalan fgalan merged commit 2bf96d0 into telefonicasc:master Apr 17, 2026
5 checks passed
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.

2 participants