Shellcheck - PR #7472 ("Add missing node for youyeetoo r1 on current kernel") #1502
Workflow file for this run
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
name: Lint on Scripts | |
run-name: 'Shellcheck - PR #${{ github.event.pull_request.number }} ("${{ github.event.pull_request.title }}")' | |
# | |
# Run ShellCheck on all scripts and generate report as build artifact | |
# | |
on: | |
workflow_dispatch: | |
pull_request: | |
types: [opened, reopened, synchronize] | |
permissions: | |
contents: read | |
concurrency: | |
group: pipeline-lint-${{github.event.pull_request.number}} | |
cancel-in-progress: true | |
jobs: | |
Shellcheck: | |
name: Shell script analysis | |
runs-on: ubuntu-latest | |
if: ${{ github.repository_owner == 'Armbian' }} | |
steps: | |
- name: Checkout repository | |
uses: actions/checkout@v4 | |
with: | |
fetch-depth: 2 | |
- name: Get changed files | |
id: changed-files | |
uses: tj-actions/changed-files@v45 | |
- name: List all changed files | |
run: | | |
# Use framework internal mechanism for checking `lib` and `extensions` code only one file is passed, | |
# and source's are followed, thus the whole project is "understood" by shellcheck. | |
# For example, when checking individual files, one variable might be thought "unused" because it | |
# is only used in another file, which does not happen when done properly. | |
bash lib/tools/shellcheck.sh | |
ret=0 | |
for file in ${{ steps.changed-files.outputs.all_changed_files }}; do | |
if [[ ! "${file}" =~ lib/|extensions/|.py|.service|.rules|.network|.netdev ]]; then | |
if grep -qE "^#\!/.*bash" $file; then | |
shellcheck --severity=error $file || ret=$? | |
fi | |
fi | |
done | |
exit $ret |