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

Arregla GHA que agrega comentario sobre entradas faltantes#3305

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
cmaureir merged 2 commits intopython:3.13fromrtobar:sparse-co-fix
Nov 23, 2024

Conversation

rtobar
Copy link
Collaborator

@rtobarrtobar commentedNov 23, 2024
edited
Loading

La acción comenzó a fallar (e.g.https://github.com/python/python-docs-es/actions/runs/11980786740/job/33405766523?pr=3304) después de mergear#3301. Esto porque la acción realiza un "sparse checkout" (para ahorrar tiempo) de ciertos archivos, y con#3301 este checkout hubiese tenido que haber incluido a todo el submódulo cpython, con lo cual se hubieran agregado varios seguidos más a la ejecución del GHA.

Este PR separa nuestras propias dependencias en un nuevo archivorequirements-own.txt que es incluido porrequirements.txt, pero que también puede ser referenciado de forma separada. El GHA en cuestión efectivamente ahora sólo instalarequirements-own.txt, evitando así referenciar al submódulo de cpython.

Por otro lado, tener nuestras propias dependencias en un archivo separado no es malo en términos de organización de la información.

Nótese que este PR en sí va a fallar, ya que el archivo .yaml definiendo el GHA fallido se toma de la rama principla, no la del PR. Pero enrtobar#9 pueden ver que el fix funciona al abrir un PR contra una rama donde el fix ya está puesto.

That will allow us to install only those if necessary (hint: we needto, sometimes).Signed-off-by: Rodrigo Tobar <rtobar@icrar.org>
We don't actually need cpython's documentation dependencies, since weonly run one of our own scripts.Signed-off-by: Rodrigo Tobar <rtobar@icrar.org>
@cmaureircmaureir merged commit70e20ec intopython:3.13Nov 23, 2024
3 of 4 checks passed
@rtobarrtobar deleted the sparse-co-fix branchNovember 27, 2024 03:42
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

2 participants
@rtobar@cmaureir

[8]ページ先頭

©2009-2025 Movatter.jp