Skip to content

Commit ea578de

Browse files
committed
Corre spell sólo sobre archivos con cambios
El chequeo ortográfico es el paso que más demora actualmente en los tests de CI (excepto por al build mismo de la documentación). Esto es, en parte, porque independiente de que un PR esté cambiando un solo archivo, el chequeo se hace sobre todo el responsitorio. Con este commit primero identificamos los archivos .po que han cambiado dentro del PR, y sólo corremos pospell sobre ellos. Signed-off-by: Rodrigo Tobar <rtobar@icrar.org>
1 parent 35071b0 commit ea578de

File tree

1 file changed

+11
-1
lines changed

1 file changed

+11
-1
lines changed

.github/workflows/main.yml

Lines changed: 11 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@ jobs:
1414
- uses: actions/checkout@v4
1515
with:
1616
submodules: 'true'
17+
fetch-depth: 2
1718
- name: Preparar Python v3.11
1819
uses: actions/setup-python@v4
1920
with:
@@ -34,6 +35,14 @@ jobs:
3435
pip list
3536
pospell --version
3637
powrap --version
38+
- name: Obtiene la lista de archivos .po con cambios
39+
id: changed-po-files
40+
uses: tj-actions/changed-files@v40
41+
with:
42+
files: |
43+
**/*.po
44+
- name: Lista los archivos .po con cambios
45+
run: echo "Archivo(s) .po con cambios:" ${{ steps.changed-po-files.outputs.all_changed_files }}
3746
- name: TRANSLATORS
3847
run: |
3948
diff -Naur TRANSLATORS <(LANG=es python scripts/sort.py < TRANSLATORS)
@@ -43,8 +52,9 @@ jobs:
4352
run: |
4453
sphinx-lint */*.po
4554
- name: Pospell
55+
if: steps.changed-po-files.outputs.test_any_changed == 'true'
4656
run: |
47-
python scripts/check_spell.py
57+
python scripts/check_spell.py ${{ steps.changed-po-files.outputs.all_changed_files }}
4858
- name: Construir documentación
4959
run: |
5060
# FIXME: Relative paths for includes in 'cpython'

0 commit comments

Comments
 (0)