Skip to content

Commit 70e20ec

Browse files
authored
Arregla GHA que agrega comentario sobre entradas faltantes (#3305)
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 archivo `requirements-own.txt` que es incluido por `requirements.txt`, pero que también puede ser referenciado de forma separada. El GHA en cuestión efectivamente ahora sólo instala `requirements-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 en rtobar#9 pueden ver que el fix funciona al abrir un PR contra una rama donde el fix ya está puesto. --------- Signed-off-by: Rodrigo Tobar <[email protected]>
1 parent 58888ac commit 70e20ec

File tree

3 files changed

+19
-19
lines changed

3 files changed

+19
-19
lines changed

.github/workflows/pr-comment.yml

+2-2
Original file line numberDiff line numberDiff line change
@@ -30,11 +30,11 @@ jobs:
3030
path: base-branch
3131
sparse-checkout-cone-mode: false
3232
sparse-checkout: |
33-
requirements.txt
33+
requirements-own.txt
3434
scripts/list_missing_entries.py
3535
- name: Instalar dependencias
3636
run: |
37-
python -m pip install -r base-branch/requirements.txt
37+
python -m pip install -r base-branch/requirements-own.txt
3838
- name: Obtiene lista de archivos con cambios
3939
id: changed-files
4040
uses: tj-actions/changed-files@v45

requirements-own.txt

+16
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,16 @@
1+
# Our own dependencies (alpha-sorted please)
2+
pip
3+
polib
4+
pospell>=1.1
5+
potodo
6+
powrap>=1.0.2
7+
pre-commit
8+
Pygments>=2.17.0
9+
PyICU
10+
setuptools
11+
sphinx-autorun
12+
sphinxemoji
13+
sphinx-intl>=2.3.0
14+
sphinx-lint==0.7.0
15+
sphinx-tabs==3.4.5
16+
tabulate

requirements.txt

+1-17
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,2 @@
11
-r cpython/Doc/requirements.txt
2-
3-
# Our own dependencies (alpha-sorted please)
4-
pip
5-
polib
6-
pospell>=1.1
7-
potodo
8-
powrap>=1.0.2
9-
pre-commit
10-
Pygments>=2.17.0
11-
PyICU
12-
setuptools
13-
sphinx-autorun
14-
sphinxemoji
15-
sphinx-intl>=2.3.0
16-
sphinx-lint==0.7.0
17-
sphinx-tabs==3.4.5
18-
tabulate
2+
-r requirements-own.txt

0 commit comments

Comments
 (0)