From 4703051902d4822a139572c12baed50f3ac23d17 Mon Sep 17 00:00:00 2001 From: Julien Palard Date: Fri, 11 Dec 2020 15:20:16 +0100 Subject: [PATCH 01/15] Trying to see if there's a list of changed files here. --- .github/workflows/tests.yml | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml index e6bf9fb7c..a8438aabe 100644 --- a/.github/workflows/tests.yml +++ b/.github/workflows/tests.yml @@ -18,6 +18,10 @@ jobs: - uses: actions/checkout@v2 - name: Register pospell problem matcher run: echo "::add-matcher::.github/problem-matchers/pospell.json" + - name: Dump GitHub event context + env: + GITHUB_EVENT_CONTEXT: ${{ toJson(github.event) }} + run: echo "$GITHUB_EVENT_CONTEXT" - name: Install pospell run: | sudo apt-get update From 4d3f59b9629bd1fd3ca86c09590a80b154523efd Mon Sep 17 00:00:00 2001 From: Julien Palard Date: Fri, 11 Dec 2020 15:47:41 +0100 Subject: [PATCH 02/15] try again --- .github/workflows/tests.yml | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml index a8438aabe..f6cf697fe 100644 --- a/.github/workflows/tests.yml +++ b/.github/workflows/tests.yml @@ -18,10 +18,13 @@ jobs: - uses: actions/checkout@v2 - name: Register pospell problem matcher run: echo "::add-matcher::.github/problem-matchers/pospell.json" - - name: Dump GitHub event context - env: - GITHUB_EVENT_CONTEXT: ${{ toJson(github.event) }} - run: echo "$GITHUB_EVENT_CONTEXT" + - uses: lots0logs/gh-action-get-changed-files@2.1.4 + id: changed_files + with: + token: ${{ secrets.GITHUB_TOKEN }} + - name: Show changed files + run: + echo '${{ steps.changed_files.outputs.all }}' - name: Install pospell run: | sudo apt-get update From 352d0c7df776bcff5c6df585cddce399344ef68d Mon Sep 17 00:00:00 2001 From: Julien Palard Date: Fri, 11 Dec 2020 15:52:14 +0100 Subject: [PATCH 03/15] Parse json --- .github/workflows/tests.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml index f6cf697fe..0aa9cf8a1 100644 --- a/.github/workflows/tests.yml +++ b/.github/workflows/tests.yml @@ -24,7 +24,7 @@ jobs: token: ${{ secrets.GITHUB_TOKEN }} - name: Show changed files run: - echo '${{ steps.changed_files.outputs.all }}' + echo '${{ join(fromJSON(steps.changed_files.outputs.all), ' ') }}' - name: Install pospell run: | sudo apt-get update From 446b1b78d474ead5546de5c6197a5dee26ac4b79 Mon Sep 17 00:00:00 2001 From: Julien Palard Date: Fri, 11 Dec 2020 15:56:40 +0100 Subject: [PATCH 04/15] Test with three files. --- .github/workflows/tests.yml | 131 ++++++++++++++++++------------------ library/aifc.po | 4 +- library/allos.po | 4 +- library/zoneinfo.po | 4 +- 4 files changed, 72 insertions(+), 71 deletions(-) diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml index 0aa9cf8a1..a4267850f 100644 --- a/.github/workflows/tests.yml +++ b/.github/workflows/tests.yml @@ -24,69 +24,70 @@ jobs: token: ${{ secrets.GITHUB_TOKEN }} - name: Show changed files run: - echo '${{ join(fromJSON(steps.changed_files.outputs.all), ' ') }}' - - name: Install pospell - run: | - sudo apt-get update - sudo apt-get install -y hunspell hunspell-fr-comprehensive - sudo python3 -m pip install pospell - - name: Run pospell - run: pospell -p dict -l fr_FR *.po */*.po + echo '${{ join(fromJSON(steps.changed_files.outputs.all), "\n") }}' + # - name: Install pospell + # run: | + # sudo apt-get update + # sudo apt-get install -y hunspell hunspell-fr-comprehensive + # sudo python3 -m pip install pospell + # - name: Run pospell + # run: pospell -p dict -l fr_FR *.po */*.po - powrap: - name: 'Longueur des lignes (powrap)' - runs-on: ubuntu-latest - steps: - - uses: actions/checkout@v2 - - name: Install powrap - run: | - sudo apt-get update - sudo apt-get install gettext - sudo python3 -m pip install powrap - - name: Run powrap - run: powrap --check --quiet *.po */*.po - - padpo: - name: 'Grammaire (padpo)' - runs-on: ubuntu-latest - steps: - - uses: actions/checkout@v2 - - uses: actions/setup-python@v2 - with: - python-version: '3.9' - - name: Install padpo - run: | - python3.9 -m venv .venv - .venv/bin/python -m pip install --upgrade pip setuptools wheel - .venv/bin/python -m pip install padpo - - name: Run padpo - run: PATH=$PATH:$PWD/.venv/bin/ padpo -i *.po */*.po 2>&1 | grep -v -Ff padpo.ignore - - sphinx: - name: 'Génération de la doc (sphinx)' - runs-on: ubuntu-latest - steps: - - uses: actions/checkout@v2 - - uses: actions/cache@v2 - with: - path: '~/cpython/' - key: '1' - - uses: actions/setup-python@v2 - with: - python-version: '3.9' - - name: Clone cpython - run: | - if [ ! -d ~/cpython ]; then - git clone https://github.com/python/cpython ~/cpython/ - else - git -C ~/cpython fetch - fi - - name: Prepare build hierarchy - run: | - mkdir -p ~/locales/fr/LC_MESSAGES/ - cp -uv --parents *.po */*.po ~/locales/fr/LC_MESSAGES/ - git -C ~/cpython checkout b30934e9afb0af3f8e2e5f0992445be775b3c630 - - name: Prepare venv - run: make -C $HOME/cpython/Doc venv - - name: sphinx-build - run: make -C $HOME/cpython/Doc SPHINXOPTS="-qW -j4 -D locale_dirs=$HOME/locales -D language=fr -D gettext_compact=0" autobuild-dev-html +# powrap: +# name: 'Longueur des lignes (powrap)' +# runs-on: ubuntu-latest +# steps: +# - uses: actions/checkout@v2 +# - name: Install powrap +# run: | +# sudo apt-get update +# sudo apt-get install gettext +# sudo python3 -m pip install powrap +# - name: Run powrap +# run: powrap --check --quiet *.po */*.po +# +# padpo: +# name: 'Grammaire (padpo)' +# runs-on: ubuntu-latest +# steps: +# - uses: actions/checkout@v2 +# - uses: actions/setup-python@v2 +# with: +# python-version: '3.9' +# - name: Install padpo +# run: | +# python3.9 -m venv .venv +# .venv/bin/python -m pip install --upgrade pip setuptools wheel +# .venv/bin/python -m pip install padpo +# - name: Run padpo +# run: PATH=$PATH:$PWD/.venv/bin/ padpo -i *.po */*.po 2>&1 | grep -v -Ff padpo.ignore +# +# sphinx: +# name: 'Génération de la doc (sphinx)' +# runs-on: ubuntu-latest +# steps: +# - uses: actions/checkout@v2 +# - uses: actions/cache@v2 +# with: +# path: '~/cpython/' +# key: '1' +# - uses: actions/setup-python@v2 +# with: +# python-version: '3.9' +# - name: Clone cpython +# run: | +# if [ ! -d ~/cpython ]; then +# git clone https://github.com/python/cpython ~/cpython/ +# else +# git -C ~/cpython fetch +# fi +# - name: Prepare build hierarchy +# run: | +# mkdir -p ~/locales/fr/LC_MESSAGES/ +# cp -uv --parents *.po */*.po ~/locales/fr/LC_MESSAGES/ +# git -C ~/cpython checkout b30934e9afb0af3f8e2e5f0992445be775b3c630 +# - name: Prepare venv +# run: make -C $HOME/cpython/Doc venv +# - name: sphinx-build +# run: make -C $HOME/cpython/Doc SPHINXOPTS="-qW -j4 -D locale_dirs=$HOME/locales -D language=fr -D gettext_compact=0" autobuild-dev-html +# diff --git a/library/aifc.po b/library/aifc.po index d1eb59206..a10b27b3f 100644 --- a/library/aifc.po +++ b/library/aifc.po @@ -6,8 +6,8 @@ msgstr "" "Project-Id-Version: Python 3\n" "Report-Msgid-Bugs-To: \n" "POT-Creation-Date: 2020-10-01 16:00+0200\n" -"PO-Revision-Date: YEAR-MO-DA HO:MI+ZONE\n" -"Last-Translator: FULL NAME \n" +"PO-Revision-Date: 2020-12-11 15:56+0100\n" +"Last-Translator: Julien Palard \n" "Language-Team: FRENCH \n" "Language: fr\n" "MIME-Version: 1.0\n" diff --git a/library/allos.po b/library/allos.po index 4fad1aeb4..20600fe50 100644 --- a/library/allos.po +++ b/library/allos.po @@ -6,8 +6,8 @@ msgstr "" "Project-Id-Version: Python 3\n" "Report-Msgid-Bugs-To: \n" "POT-Creation-Date: 2020-08-24 09:01+0200\n" -"PO-Revision-Date: 2018-07-27 22:58+0200\n" -"Last-Translator: FULL NAME \n" +"PO-Revision-Date: 2020-12-11 15:56+0100\n" +"Last-Translator: Julien Palard \n" "Language-Team: FRENCH \n" "Language: fr\n" "MIME-Version: 1.0\n" diff --git a/library/zoneinfo.po b/library/zoneinfo.po index 7292f7728..17e850a7d 100644 --- a/library/zoneinfo.po +++ b/library/zoneinfo.po @@ -6,8 +6,8 @@ msgstr "" "Project-Id-Version: Python 3\n" "Report-Msgid-Bugs-To: \n" "POT-Creation-Date: 2020-08-24 09:01+0200\n" -"PO-Revision-Date: 2020-07-20 15:07+0200\n" -"Last-Translator: FULL NAME \n" +"PO-Revision-Date: 2020-12-11 15:55+0100\n" +"Last-Translator: Julien Palard \n" "Language-Team: FRENCH \n" "Language: fr\n" "MIME-Version: 1.0\n" From 24e1b97db12d686ef0edbe72058c0a142ef74888 Mon Sep 17 00:00:00 2001 From: Julien Palard Date: Fri, 11 Dec 2020 15:58:21 +0100 Subject: [PATCH 05/15] GH syntax? --- .github/workflows/tests.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml index a4267850f..ea223d422 100644 --- a/.github/workflows/tests.yml +++ b/.github/workflows/tests.yml @@ -24,7 +24,7 @@ jobs: token: ${{ secrets.GITHUB_TOKEN }} - name: Show changed files run: - echo '${{ join(fromJSON(steps.changed_files.outputs.all), "\n") }}' + echo '${{ join(fromJSON(steps.changed_files.outputs.all), '\n') }}' # - name: Install pospell # run: | # sudo apt-get update From 27e1dc7f7cb9c168c82db96965b0df3a3070daa2 Mon Sep 17 00:00:00 2001 From: Julien Palard Date: Fri, 11 Dec 2020 16:05:20 +0100 Subject: [PATCH 06/15] Find changed po files. --- .github/workflows/tests.yml | 22 ++++++++++++---------- 1 file changed, 12 insertions(+), 10 deletions(-) diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml index ea223d422..37b1b526c 100644 --- a/.github/workflows/tests.yml +++ b/.github/workflows/tests.yml @@ -22,16 +22,18 @@ jobs: id: changed_files with: token: ${{ secrets.GITHUB_TOKEN }} - - name: Show changed files - run: - echo '${{ join(fromJSON(steps.changed_files.outputs.all), '\n') }}' - # - name: Install pospell - # run: | - # sudo apt-get update - # sudo apt-get install -y hunspell hunspell-fr-comprehensive - # sudo python3 -m pip install pospell - # - name: Run pospell - # run: pospell -p dict -l fr_FR *.po */*.po + - name: Install pospell + run: | + sudo apt-get update + sudo apt-get install -y hunspell hunspell-fr-comprehensive + sudo python3 -m pip install pospell + - name: Run pospell + with: + ADDED_FILES: ${{ join(fromJSON(steps.changed_files.outputs.added), ' ') }} + MODIFIED_FILES: ${{ join(fromJSON(steps.changed_files.outputs.modified), ' ') }} + run: | + CHANGED_PO_FILES=$(printf "%s %s\n" "$ADDED_FILES" "$MODIFIED_FILES" | tr ' ' '\n' | grep '.po$') + [ -n "$CHANGED_PO_FILES" ] && pospell -p dict -l fr_FR $CHANGED_PO_FILES # powrap: # name: 'Longueur des lignes (powrap)' From ff76de7f01fe0034670faa81e09795502e3a1df6 Mon Sep 17 00:00:00 2001 From: Julien Palard Date: Fri, 11 Dec 2020 16:06:58 +0100 Subject: [PATCH 07/15] maybe --- .github/workflows/tests.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml index 37b1b526c..904b1bff4 100644 --- a/.github/workflows/tests.yml +++ b/.github/workflows/tests.yml @@ -29,8 +29,8 @@ jobs: sudo python3 -m pip install pospell - name: Run pospell with: - ADDED_FILES: ${{ join(fromJSON(steps.changed_files.outputs.added), ' ') }} - MODIFIED_FILES: ${{ join(fromJSON(steps.changed_files.outputs.modified), ' ') }} + ADDED_FILES: '${{ join(fromJSON(steps.changed_files.outputs.added), '' '') }}' + MODIFIED_FILES: '${{ join(fromJSON(steps.changed_files.outputs.modified), '' '') }}' run: | CHANGED_PO_FILES=$(printf "%s %s\n" "$ADDED_FILES" "$MODIFIED_FILES" | tr ' ' '\n' | grep '.po$') [ -n "$CHANGED_PO_FILES" ] && pospell -p dict -l fr_FR $CHANGED_PO_FILES From a5e5e61e697dc706723904c9e54387ce7a804b76 Mon Sep 17 00:00:00 2001 From: Julien Palard Date: Fri, 11 Dec 2020 16:07:57 +0100 Subject: [PATCH 08/15] Oh it's env here. --- .github/workflows/tests.yml | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml index 904b1bff4..81493ee72 100644 --- a/.github/workflows/tests.yml +++ b/.github/workflows/tests.yml @@ -28,9 +28,9 @@ jobs: sudo apt-get install -y hunspell hunspell-fr-comprehensive sudo python3 -m pip install pospell - name: Run pospell - with: - ADDED_FILES: '${{ join(fromJSON(steps.changed_files.outputs.added), '' '') }}' - MODIFIED_FILES: '${{ join(fromJSON(steps.changed_files.outputs.modified), '' '') }}' + env: + ADDED_FILES: ${{ join(fromJSON(steps.changed_files.outputs.added), ' ') }} + MODIFIED_FILES: ${{ join(fromJSON(steps.changed_files.outputs.modified), ' ') }} run: | CHANGED_PO_FILES=$(printf "%s %s\n" "$ADDED_FILES" "$MODIFIED_FILES" | tr ' ' '\n' | grep '.po$') [ -n "$CHANGED_PO_FILES" ] && pospell -p dict -l fr_FR $CHANGED_PO_FILES From 42a750cd9eb945e6ee0bba41ee51bba3b154af19 Mon Sep 17 00:00:00 2001 From: Julien Palard Date: Fri, 11 Dec 2020 16:21:32 +0100 Subject: [PATCH 09/15] Playing with it. --- .github/workflows/tests.yml | 125 +++++++++++++++++------------------- 1 file changed, 58 insertions(+), 67 deletions(-) diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml index 81493ee72..9d420e979 100644 --- a/.github/workflows/tests.yml +++ b/.github/workflows/tests.yml @@ -11,85 +11,76 @@ on: - '*.*' jobs: - pospell: - name: 'Orthographe (pospell)' + Vérifications: + strategy: + matrix: + tool: + - name: Orthographe + package: pospell + apt_dependencies: hunspell hunspell-fr-comprehensive + command: pospell -p dict -l fr_FR + - name: Longueur des lignes + package: powrap + apt_dependencies: gettext + command: powrap --check --quiet --diff + - name: Grammaire + package: padpo + command: 'padpo -i 2>&1' + command_suffix: '| grep -v -Ff padpo.ignore' + + name: ${{ matrix.tool.name }} (${{ matrix.tool.package }}) runs-on: ubuntu-latest steps: - uses: actions/checkout@v2 - - name: Register pospell problem matcher + - name: Register problem matchers run: echo "::add-matcher::.github/problem-matchers/pospell.json" - uses: lots0logs/gh-action-get-changed-files@2.1.4 id: changed_files with: token: ${{ secrets.GITHUB_TOKEN }} - - name: Install pospell + - uses: actions/setup-python@v2 + with: + python-version: '3.9' + - name: Install ${{ matrix.tool.package }} run: | - sudo apt-get update - sudo apt-get install -y hunspell hunspell-fr-comprehensive - sudo python3 -m pip install pospell - - name: Run pospell + [ -n "${{ apt_dependencies }}" ] && sudo apt-get update + [ -n "${{ apt_dependencies }}" ] && sudo apt-get install -y ${{ apt_dependencies }} + python -m pip install --upgrade pip setuptools wheel + python -m pip install ${{ matrix.tool.package }} + - name: Run ${{ matrix.tool.package }} env: ADDED_FILES: ${{ join(fromJSON(steps.changed_files.outputs.added), ' ') }} MODIFIED_FILES: ${{ join(fromJSON(steps.changed_files.outputs.modified), ' ') }} run: | CHANGED_PO_FILES=$(printf "%s %s\n" "$ADDED_FILES" "$MODIFIED_FILES" | tr ' ' '\n' | grep '.po$') - [ -n "$CHANGED_PO_FILES" ] && pospell -p dict -l fr_FR $CHANGED_PO_FILES + printf "Running on" "$CHANGED_PO_FILES" + [ -n "$CHANGED_PO_FILES" ] && ${{ matrix.tool.command }} $CHANGED_PO_FILES ${{ matrix.tool.command_suffix }} -# powrap: -# name: 'Longueur des lignes (powrap)' -# runs-on: ubuntu-latest -# steps: -# - uses: actions/checkout@v2 -# - name: Install powrap -# run: | -# sudo apt-get update -# sudo apt-get install gettext -# sudo python3 -m pip install powrap -# - name: Run powrap -# run: powrap --check --quiet *.po */*.po -# -# padpo: -# name: 'Grammaire (padpo)' -# runs-on: ubuntu-latest -# steps: -# - uses: actions/checkout@v2 -# - uses: actions/setup-python@v2 -# with: -# python-version: '3.9' -# - name: Install padpo -# run: | -# python3.9 -m venv .venv -# .venv/bin/python -m pip install --upgrade pip setuptools wheel -# .venv/bin/python -m pip install padpo -# - name: Run padpo -# run: PATH=$PATH:$PWD/.venv/bin/ padpo -i *.po */*.po 2>&1 | grep -v -Ff padpo.ignore -# -# sphinx: -# name: 'Génération de la doc (sphinx)' -# runs-on: ubuntu-latest -# steps: -# - uses: actions/checkout@v2 -# - uses: actions/cache@v2 -# with: -# path: '~/cpython/' -# key: '1' -# - uses: actions/setup-python@v2 -# with: -# python-version: '3.9' -# - name: Clone cpython -# run: | -# if [ ! -d ~/cpython ]; then -# git clone https://github.com/python/cpython ~/cpython/ -# else -# git -C ~/cpython fetch -# fi -# - name: Prepare build hierarchy -# run: | -# mkdir -p ~/locales/fr/LC_MESSAGES/ -# cp -uv --parents *.po */*.po ~/locales/fr/LC_MESSAGES/ -# git -C ~/cpython checkout b30934e9afb0af3f8e2e5f0992445be775b3c630 -# - name: Prepare venv -# run: make -C $HOME/cpython/Doc venv -# - name: sphinx-build -# run: make -C $HOME/cpython/Doc SPHINXOPTS="-qW -j4 -D locale_dirs=$HOME/locales -D language=fr -D gettext_compact=0" autobuild-dev-html -# + sphinx: + name: 'Génération de la doc (sphinx)' + runs-on: ubuntu-latest + steps: + - uses: actions/checkout@v2 + - uses: actions/cache@v2 + with: + path: '~/cpython/' + key: '1' + - uses: actions/setup-python@v2 + with: + python-version: '3.9' + - name: Clone cpython + run: | + if [ ! -d ~/cpython ]; then + git clone https://github.com/python/cpython ~/cpython/ + else + git -C ~/cpython fetch + fi + - name: Prepare build hierarchy + run: | + mkdir -p ~/locales/fr/LC_MESSAGES/ + cp -uv --parents *.po */*.po ~/locales/fr/LC_MESSAGES/ + git -C ~/cpython checkout b30934e9afb0af3f8e2e5f0992445be775b3c630 + - name: Prepare venv + run: make -C $HOME/cpython/Doc venv + - name: sphinx-build + run: make -C $HOME/cpython/Doc SPHINXOPTS="-qW -j4 -D locale_dirs=$HOME/locales -D language=fr -D gettext_compact=0" autobuild-dev-html From 07059012b2999b8a3abbb5cd0e4866e337645e0e Mon Sep 17 00:00:00 2001 From: Julien Palard Date: Fri, 11 Dec 2020 16:22:43 +0100 Subject: [PATCH 10/15] oops --- .github/workflows/tests.yml | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml index 9d420e979..34baff106 100644 --- a/.github/workflows/tests.yml +++ b/.github/workflows/tests.yml @@ -43,8 +43,7 @@ jobs: python-version: '3.9' - name: Install ${{ matrix.tool.package }} run: | - [ -n "${{ apt_dependencies }}" ] && sudo apt-get update - [ -n "${{ apt_dependencies }}" ] && sudo apt-get install -y ${{ apt_dependencies }} + [ -n "${{ matrix.tool.apt_dependencies }}" ] && sudo apt-get update && sudo apt-get install -y ${{ matrix.tool.apt_dependencies }} python -m pip install --upgrade pip setuptools wheel python -m pip install ${{ matrix.tool.package }} - name: Run ${{ matrix.tool.package }} From 6eaeee686b9412a585599fb1c9fd48e93206e626 Mon Sep 17 00:00:00 2001 From: Julien Palard Date: Fri, 11 Dec 2020 16:25:08 +0100 Subject: [PATCH 11/15] Pas d'accents ici --- .github/workflows/tests.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml index 34baff106..67b244a1b 100644 --- a/.github/workflows/tests.yml +++ b/.github/workflows/tests.yml @@ -11,7 +11,7 @@ on: - '*.*' jobs: - Vérifications: + checks: strategy: matrix: tool: From 58df8dcaefed960c200434068340148437bcbc7f Mon Sep 17 00:00:00 2001 From: Julien Palard Date: Fri, 11 Dec 2020 16:31:02 +0100 Subject: [PATCH 12/15] Debugging, debugging, debugging... --- .github/workflows/tests.yml | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml index 67b244a1b..aabfc2cd1 100644 --- a/.github/workflows/tests.yml +++ b/.github/workflows/tests.yml @@ -51,6 +51,10 @@ jobs: ADDED_FILES: ${{ join(fromJSON(steps.changed_files.outputs.added), ' ') }} MODIFIED_FILES: ${{ join(fromJSON(steps.changed_files.outputs.modified), ' ') }} run: | + echo "$MODIFIED_FILES" + printf "%s %s\n" "$ADDED_FILES" "$MODIFIED_FILES" + printf "%s %s\n" "$ADDED_FILES" "$MODIFIED_FILES" | tr ' ' '\n' + printf "%s %s\n" "$ADDED_FILES" "$MODIFIED_FILES" | tr ' ' '\n' | grep '.po$' CHANGED_PO_FILES=$(printf "%s %s\n" "$ADDED_FILES" "$MODIFIED_FILES" | tr ' ' '\n' | grep '.po$') printf "Running on" "$CHANGED_PO_FILES" [ -n "$CHANGED_PO_FILES" ] && ${{ matrix.tool.command }} $CHANGED_PO_FILES ${{ matrix.tool.command_suffix }} From 771a9d43100c00fe76d3c48acdda546ebdfa7913 Mon Sep 17 00:00:00 2001 From: Julien Palard Date: Fri, 11 Dec 2020 16:34:42 +0100 Subject: [PATCH 13/15] =?UTF-8?q?Gn=C3=A9?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .github/workflows/tests.yml | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml index aabfc2cd1..5b8224d34 100644 --- a/.github/workflows/tests.yml +++ b/.github/workflows/tests.yml @@ -51,12 +51,9 @@ jobs: ADDED_FILES: ${{ join(fromJSON(steps.changed_files.outputs.added), ' ') }} MODIFIED_FILES: ${{ join(fromJSON(steps.changed_files.outputs.modified), ' ') }} run: | - echo "$MODIFIED_FILES" - printf "%s %s\n" "$ADDED_FILES" "$MODIFIED_FILES" - printf "%s %s\n" "$ADDED_FILES" "$MODIFIED_FILES" | tr ' ' '\n' printf "%s %s\n" "$ADDED_FILES" "$MODIFIED_FILES" | tr ' ' '\n' | grep '.po$' CHANGED_PO_FILES=$(printf "%s %s\n" "$ADDED_FILES" "$MODIFIED_FILES" | tr ' ' '\n' | grep '.po$') - printf "Running on" "$CHANGED_PO_FILES" + printf "Running on %s\n" "$CHANGED_PO_FILES" [ -n "$CHANGED_PO_FILES" ] && ${{ matrix.tool.command }} $CHANGED_PO_FILES ${{ matrix.tool.command_suffix }} sphinx: From 04bf0f7e2730b2a3bf08beeab2372aef40705942 Mon Sep 17 00:00:00 2001 From: Julien Palard Date: Fri, 11 Dec 2020 16:38:25 +0100 Subject: [PATCH 14/15] Proper display --- .github/workflows/tests.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml index 5b8224d34..2abfd7ff7 100644 --- a/.github/workflows/tests.yml +++ b/.github/workflows/tests.yml @@ -53,7 +53,7 @@ jobs: run: | printf "%s %s\n" "$ADDED_FILES" "$MODIFIED_FILES" | tr ' ' '\n' | grep '.po$' CHANGED_PO_FILES=$(printf "%s %s\n" "$ADDED_FILES" "$MODIFIED_FILES" | tr ' ' '\n' | grep '.po$') - printf "Running on %s\n" "$CHANGED_PO_FILES" + echo "Running on:" $CHANGED_PO_FILES [ -n "$CHANGED_PO_FILES" ] && ${{ matrix.tool.command }} $CHANGED_PO_FILES ${{ matrix.tool.command_suffix }} sphinx: From 045a2fe774e61cd3c6c9b610013ba0a8fb47fdc7 Mon Sep 17 00:00:00 2001 From: Julien Palard Date: Fri, 11 Dec 2020 16:38:39 +0100 Subject: [PATCH 15/15] removing debug --- .github/workflows/tests.yml | 1 - 1 file changed, 1 deletion(-) diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml index 2abfd7ff7..6e9d7f726 100644 --- a/.github/workflows/tests.yml +++ b/.github/workflows/tests.yml @@ -51,7 +51,6 @@ jobs: ADDED_FILES: ${{ join(fromJSON(steps.changed_files.outputs.added), ' ') }} MODIFIED_FILES: ${{ join(fromJSON(steps.changed_files.outputs.modified), ' ') }} run: | - printf "%s %s\n" "$ADDED_FILES" "$MODIFIED_FILES" | tr ' ' '\n' | grep '.po$' CHANGED_PO_FILES=$(printf "%s %s\n" "$ADDED_FILES" "$MODIFIED_FILES" | tr ' ' '\n' | grep '.po$') echo "Running on:" $CHANGED_PO_FILES [ -n "$CHANGED_PO_FILES" ] && ${{ matrix.tool.command }} $CHANGED_PO_FILES ${{ matrix.tool.command_suffix }}