Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Actualizar la documentación para indicar que el clon debe ser recursivo#3353

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
rtobar merged 6 commits intopython:3.13fromkbiggers:issue-3352
Jan 27, 2025

Conversation

kbiggers
Copy link
Contributor

@kbiggerskbiggers commentedDec 28, 2024
edited
Loading

Actualizar la documentación para indicar que el clon debe ser recursivo

closes#3352

@github-actionsGitHub Actions
Copy link

All entries translated, horray! 🎉

@rtobar
Copy link
Collaborator

@kbiggers los cambios de este PR están basados en tu branch donde tradujiste lexical_analysis.po. ¿Puedes hacer un rebase sobre la rama 3.13? ¡Gracias!

En cuanto a los cambios mismo, si bien son correctos, creo que no son óptimos. Hacer el clone recursivo es mucho más costoso que hacer el clone sólo de python-docs-es, y para lo único que se necesita el submódulo de cpython es para construir la documentación -- que creo que muchos usuarios no realizarn de forma local. Por otro lado, cuando uno hacemake build el submódulo se inicializa si no está inicializado. Dado todo esto,, creo que tienen más sentido otras alternativas dondeno se hace un clon recursivo:

  1. Cambiar las instrucciones para que el comandopip install indique que hay que instalarpip install requirements-own.txt en vez derequirements.txt. Ese sería un cambio bien simple, ya que sabemos que el resto de la infraestructura ya funciona con los archivosrequirements*.txt tal como están.
  2. Renombrarrequirements.txt arequirements-build.txt yrequirements-own.txt arequirements.txt. Así queda claro que el primero contiene las dependencias necesarias para hacer el build, mientras que el segundo, más implícitamente, contiene las dependencias que se necesitan para hacer la traducción. Dados los cambios de nombres, se requeriría un par de cambios en los jobs de CI tal vez, y en elMakefile, pero nada muy complejo.

Personalmente yo me decantaría por la primera opción, pero sería bueno escuchar otras opiniones. Y, en primer lugar, saber si ésta es una opción válida en primer lugar.

@kbiggers
Copy link
ContributorAuthor

Gracias por la respuesta@rtobar! Coincido contigo que la opción 1 parece la más simple, y la mejor por ahora. Tal vez podemos añadir algún comentario extra a la guía sobre como construir la documentación de forma local (por si acaso algún usuario quiere hacerlo)?

kbiggersand others added6 commitsJanuary 26, 2025 21:34
Co-authored-by: rtobar <rtobarc@gmail.com>
Co-authored-by: rtobar <rtobarc@gmail.com>
…-own.txt. Añadir alguna nota extra sobre los pasos para construir la documentación de una manera local
…a menos que estamos haciendo una construcción¨
@rtobarrtobar merged commit8144640 intopython:3.13Jan 27, 2025
4 checks passed
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@cmaureircmaureircmaureir left review comments

@rtobarrtobarrtobar left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Actualizar la documentación ahora que dependemos de cpython en requirements.txt
3 participants
@kbiggers@rtobar@cmaureir

[8]ページ先頭

©2009-2025 Movatter.jp