Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(code_layout): use env var and pipe to bypass the "Argument list… #42

Open
wants to merge 12 commits into
base: master
Choose a base branch
from
25 changes: 0 additions & 25 deletions .github/workflows/backport.yml

This file was deleted.

184 changes: 0 additions & 184 deletions .github/workflows/build-docker.yml

This file was deleted.

110 changes: 0 additions & 110 deletions .github/workflows/build_artifact_comment.yml

This file was deleted.

44 changes: 30 additions & 14 deletions .github/workflows/code_layout.yml
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ jobs:
- name: Set up Python 3.10
uses: actions/setup-python@v5
with:
python-version: '3.10'
python-version: "3.10"
- name: Install requirements
run: |
wget https://www.doxygen.nl/files/doxygen-${DOXYGEN_VERSION}.linux.bin.tar.gz
Expand Down Expand Up @@ -65,8 +65,8 @@ jobs:
uses: actions/checkout@v4
- name: Install Requirements
run: |
sudo apt install -y \
shellcheck
sudo apt install -y \
shellcheck

- name: Run Shellcheck
run: ./tests/code_layout/test_shellcheck.sh
Expand Down Expand Up @@ -154,25 +154,41 @@ jobs:
steps:
- name: Checkout
uses: actions/checkout@v4

- name: Install Requirements
run: |
sudo apt install -y \
expect \
silversearcher-ag
- uses: tj-actions/changed-files@v42

- name: Retrieve changed files
uses: tj-actions/changed-files@v43
id: changed_files
with:
separator: ' '
files_ignore: |
**/*.sip.in
**/*.png
**/*.svg
**/*.tif*
**/*.gpkg
**/*.qgz
**/*.qgs
separator: " "
# files_ignore: |
# **/*.sip.in
# **/*.png
# **/*.svg
# **/*.tif*
# **/*.gpkg
# **/*.qgz
# **/*.qgs

- name: Echo list
if: steps.changed_files.outputs.any_changed == 'true'
env:
ALL_CHANGED_FILES: ${{ steps.changed_files.outputs.all_changed_files }}
run: |
echo "=============================================================================" >/dev/stderr \
&& echo "$ALL_CHANGED_FILES" >/dev/stderr \
&& echo "=============================================================================" >/dev/stderr

- name: Spell Test
run: ./scripts/spell_check/check_spelling.sh -r "${{ steps.changed_files.outputs.all_changed_files }}"
if: steps.changed_files.outputs.any_changed == 'true'
env:
ALL_CHANGED_FILES: ${{ steps.changed_files.outputs.all_changed_files }}
run: echo "$ALL_CHANGED_FILES" | ./scripts/spell_check/check_spelling.sh

sip_check:
runs-on: ubuntu-latest
Expand Down
Loading
Loading