Update actions/checkout version on README.md (#101)

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Joakim Sørensen <hi@ludeeus.dev>

.devcontainer.json

@@ -0,0 +1,4 @@
+{
+	"name": "ludeeus/action-shellcheck",
+	"image": "mcr.microsoft.com/devcontainers/base:bullseye"
+}

.github/dependabot.yml

@@ -0,0 +1,8 @@
+version: 2
+updates:
+  - package-ecosystem: "github-actions"
+    directory: "/"
+    schedule:
+      interval: weekly
+      time: "06:00"
+    open-pull-requests-limit: 10

.github/problem-matcher.json

@@ -1,22 +0,0 @@
-{
-    "problemMatcher": [
-      {
-        "owner": "shellcheck",
-        "pattern": [
-          {
-              "regexp": "^In\\s(.+)\\sline\\s(\\d+):$",
-              "file": 1,
-              "line": 2
-          },
-          {
-            "regexp": ".*"
-          },
-          {
-              "regexp": "(SC\\d+):\\s(.+)$",
-              "code": 1,
-              "message": 2
-          }
-      ]
-      }
-    ]
-  }

.github/workflows/additional_files.yml

@@ -2,8 +2,13 @@ name: 'additional_files'
 
 on: 
   push:
-    branches: ["master"]
+    branches:
+      - "master"
   pull_request:
+    branches:
+      - "master"
+
+permissions: {}
 
 jobs:
   additional_files:
@@ -16,14 +21,14 @@ jobs:
           - macos-latest
     steps:
     - name: Checkout
-      uses: actions/checkout@v2
+      uses: actions/checkout@v4.1.0
 
     - name: Run ShellCheck
       uses: ./
       id: check
       with:
         additional_files: run finish discovery
-        ignore: ignore
+        ignore_paths: ignore
         scandir: testfiles
 
     - name: Verify check

.github/workflows/check_together.yml

@@ -2,8 +2,13 @@ name: 'check_together'
 
 on: 
   push:
-    branches: ["master"]
+    branches:
+      - "master"
   pull_request:
+    branches:
+      - "master"
+
+permissions: {}
 
 jobs:
   check_together:
@@ -16,13 +21,13 @@ jobs:
           - macos-latest
     steps:
     - name: Checkout
-      uses: actions/checkout@v2
+      uses: actions/checkout@v4.1.0
 
     - name: Run ShellCheck
       uses: ./
       id: check
       with:
-        ignore: ignore
+        ignore_paths: ignore
         check_together: true
 
     - name: Verify check

.github/workflows/ignore_names.yml

@@ -1,13 +1,18 @@
-name: 'base'
+name: 'ignore_names'
 
 on: 
   push:
-    branches: ["master"]
+    branches:
+      - "master"
   pull_request:
+    branches:
+      - "master"
+
+permissions: {}
 
 jobs:
-  base:
-    name: base
+  ignore_names:
+    name: ignore_names
     runs-on: ${{ matrix.os }}
     strategy:
       matrix:
@@ -16,18 +21,19 @@ jobs:
           - macos-latest
     steps:
     - name: Checkout
-      uses: actions/checkout@v2
+      uses: actions/checkout@v4.1.0
 
     - name: Run ShellCheck
       uses: ./
       id: check
       with:
-        ignore: ignore
+        ignore_paths: ignore
+        ignore_names: ignore_single_file.sh
 
     - name: Verify check
       run: | 
         expect="testfiles/test.bash"
-        notexpect="testfiles/ignore/ignore.bash"
+        notexpect="testfiles/ignore_single_file.sh"
 
         if [[ ! "${{ steps.check.outputs.files }}" =~ $expect ]];then
           echo "::error:: Expected file $expect not found in ${{ steps.check.outputs.files }}"
@@ -35,4 +41,4 @@ jobs:
         elif [[ "${{ steps.check.outputs.files }}" =~ $notexpect ]];then
           echo "::error:: Expected file $notexpect found in ${{ steps.check.outputs.files }}"
           exit 1
-        fi 
+        fi 

.github/workflows/ignore_paths.yml

@@ -0,0 +1,74 @@
+name: "ignore_paths"
+
+on:
+  push:
+    branches:
+      - "master"
+  pull_request:
+    branches:
+      - "master"
+
+permissions: {}
+
+jobs:
+  ignore_paths:
+    name: ignore_paths
+    runs-on: ${{ matrix.os }}
+    strategy:
+      matrix:
+        os:
+          - ubuntu-latest
+          - macos-latest
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4.1.0
+
+      - name: Run ShellCheck
+        uses: ./
+        id: check
+        with:
+          ignore_paths: ignore ./testfiles/ignore_some/duplicate_name.bash **/ignore_some/ignore.bash
+
+      - name: Verify check
+        run: |
+          fail=false
+
+          # verify a non-ignored path is not excluded
+          expect="testfiles/test.bash"
+          if [[ ! "${{ steps.check.outputs.files }}" =~ $expect ]];then
+            echo "::error:: Expected file $expect not found in ${{ steps.check.outputs.files }}"
+            fail=true
+          fi
+
+          # verify a file with the same name as an ignored file but at a
+          # different path is not excluded
+          expect="testfiles/duplicate_name.bash"
+          if [[ ! "${{ steps.check.outputs.files }}" =~ $expect ]];then
+            echo "::error:: Expected file $expect not found in ${{ steps.check.outputs.files }}"
+            fail=true
+          fi
+
+          # verify ignored full path excluded
+          notexpect="testfiles/ignore_some/duplicate_name.bash"
+          if [[ "${{ steps.check.outputs.files }}" =~ $notexpect ]];then
+            echo "::error:: Unexpected file $notexpect found in ${{ steps.check.outputs.files }}"
+            fail=true
+          fi
+
+          # verify ignored directory excluded
+          notexpect="testfiles/ignore/ignore.bash"
+          if [[ "${{ steps.check.outputs.files }}" =~ $notexpect ]];then
+            echo "::error:: Unexpected file $notexpect found in ${{ steps.check.outputs.files }}"
+            fail=true
+          fi
+
+          # verify ignored glob excluded
+          notexpect="testfiles/ignore_some/ignore.bash"
+          if [[ "${{ steps.check.outputs.files }}" =~ $notexpect ]];then
+            echo "::error:: Unexpected file $notexpect found in ${{ steps.check.outputs.files }}"
+            fail=true
+          fi
+
+          if $fail;then
+            exit 1
+          fi

.github/workflows/scandir.yml

@@ -1,9 +1,14 @@
 name: 'scandir'
 
-on: 
+on:
   push:
-    branches: ["master"]
+    branches:
+      - "master"
   pull_request:
+    branches:
+      - "master"
+
+permissions: {}
 
 jobs:
   scandir:
@@ -16,7 +21,7 @@ jobs:
           - macos-latest
     steps:
     - name: Checkout
-      uses: actions/checkout@v2
+      uses: actions/checkout@v4.1.0
 
     - name: Run ShellCheck
       uses: ./
@@ -25,8 +30,8 @@ jobs:
         scandir: testfiles/scandir
 
     - name: Verify check
-      run: | 
-        expect="testfiles/scandir/test.bash"
+      run: |
+        expect="testfiles/scandir/run[[:space:]]me.bash"
         notexpect="testfiles/test.bash"
 
         if [[ ! "${{ steps.one.outputs.files }}" =~ $expect ]];then
@@ -35,17 +40,17 @@ jobs:
         elif [[ "${{ steps.one.outputs.files }}" =~ $notexpect ]];then
           echo "::error:: Expected file $notexpect found in ${{ steps.one.outputs.files }}"
           exit 1
-        fi 
+        fi
 
     - name: Run ShellCheck
       uses: ./
       id: two
       with:
         scandir: './testfiles/scandir'
-        ignore: ignore
+        ignore_paths: ignore
 
     - name: Verify check
-      run: | 
+      run: |
         expect="testfiles/scandir/test.bash"
         notexpect="testfiles/test.bash"
 
@@ -55,4 +60,4 @@ jobs:
         elif [[ "${{ steps.two.outputs.files }}" =~ $notexpect ]];then
           echo "::error:: Expected file $notexpect found in ${{ steps.two.outputs.files }}"
           exit 1
-        fi 
+        fi

README.md

@@ -1,30 +1,36 @@
 # ShellCheck
 
-_GitHub action for ShellCheck._
+_GitHub action for [ShellCheck](https://www.shellcheck.net/)._
 
 ## Example
 
 ```yaml
 on:
   push:
-    branch:
+    branches:
       - master
 
-name: 'Trigger: Push action'
+name: "Trigger: Push action"
+permissions: {}
 
 jobs:
   shellcheck:
     name: Shellcheck
     runs-on: ubuntu-latest
     steps:
-    - uses: actions/checkout@v2
-    - name: Run ShellCheck
-      uses: ludeeus/action-shellcheck@master
+      - uses: actions/checkout@v4
+      - name: Run ShellCheck
+        uses: ludeeus/action-shellcheck@master
 ```
 
-## Globally disable checks
+## ShellCheck options
 
-To disable specific checks add it to a `SHELLCHECK_OPTS` env key in the job definition.
+You can pass any supported ShellCheck option or flag with the `SHELLCHECK_OPTS` env key in the job definition.
+
+Some examples include:
+
+- To disable specific checks (eg: `-e SC2059 -e SC2034 -e SC1090`)
+- To test against different shells (eg: `-s dash` or `-s ksh`)
 
 example:
 
@@ -36,14 +42,18 @@ example:
         SHELLCHECK_OPTS: -e SC2059 -e SC2034 -e SC1090
 ```
 
-## Ignore paths
+## Ignore paths and names
 
-You can use the `ignore` input to disable specific directories.
+You can use the `ignore_paths` and `ignore_names` input to disable specific directories and files.
+These are passed as environment variables, and should evaluate to a single space-separated string.
+It may be convenient to use [`>-`](https://yaml.org/spec/1.2.2/#65-line-folding) for readability if you have multiple selectors.
 
 ```text
 sample structure:
-sample/directory/with/files/toignore/test.sh
+sample/directory/with/files/ignoreme/test.sh
+sample/directory/with/files/ignoremetoo/test.sh
 sample/directory/with/files/test.sh
+sample/directory/with/files/ignorable.sh
 ```
 
 example:
@@ -53,10 +63,27 @@ example:
     - name: Run ShellCheck
       uses: ludeeus/action-shellcheck@master
       with:
-        ignore: toignore
+        ignore_paths: >-
+          ignoreme
+          ignoremetoo
+        ignore_names: ignorable.sh
 ```
 
-This will skip `sample/directory/with/files/toignore/test.sh`
+This will skip `sample/directory/with/files/ignoreme/test.sh`, `sample/directory/with/files/ignoremetoo/test.sh` and `sample/directory/with/files/ignorable.sh`.
+
+You can also ignore specific files using full paths or glob patterns with `ignore_paths`.
+
+example:
+
+```yaml
+    ...
+    - name: Run ShellCheck
+      uses: ludeeus/action-shellcheck@master
+      with:
+        ignore_paths: ./sample/directory/with/files/ignorable.sh **/ignoreme/test.sh
+```
+
+This will skip `sample/directory/with/files/ignorable.sh` and `sample/directory/with/files/ignoreme/test.sh`.
 
 ## Minimum severity of errors to consider (error, warning, info, style)
 
@@ -113,15 +140,29 @@ If you need to scan for unusual files, you can use the `additional_files` key.
        additional_files: 'run finish'
 ```
 
-## Disable problem matcher
+## Change output format
 
-If you do not want to have the problem-matcher annotate files, you can disable it
-by setting `disable_matcher` to `true`.
+Shellcheck can print output in these formats: `checkstyle`, `diff`, `gcc`, `json`, `json1`, `quiet`, `tty`. See some examples [here](https://github.com/koalaman/shellcheck/wiki/Integration#pick-the-output-format-that-makes-your-life-easier).
+
+- `tty` has multi-line log messages
+- `gcc` has single-line log messages
 
 ```yaml
    ...
    - name: Run ShellCheck
      uses: ludeeus/action-shellcheck@master
      with:
-       disable_matcher: true
-```
+       format: tty
+```
+
+## Run a specific version of Shellcheck
+
+If running the latest stable version of Shellcheck is not to your liking, you can specify a concrete version of Shellcheck to be used. When specifying a custom version, please use any of the released versions listed in the [Shellcheck repository](https://github.com/koalaman/shellcheck/tags).
+
+```yaml
+   ...
+   - name: Run ShellCheck
+     uses: ludeeus/action-shellcheck@master
+     with:
+       version: v0.9.0
+```

action.yaml

@@ -3,13 +3,22 @@ author: "Ludeeus <hi@ludeeus.dev>"
 description: "GitHub action for ShellCheck."
 inputs:
   additional_files:
-    description: "A space seperated list of additional filename to check"
+    description: "A space separated list of additional filename to check"
     required: false
     default: ""
   ignore:
     description: "Paths to ignore when running ShellCheck"
     required: false
     default: ""
+    deprecationMessage: "Use ignore_paths or ignore_names instead."
+  ignore_paths:
+    description: "Paths to ignore when running ShellCheck"
+    required: false
+    default: ""
+  ignore_names:
+    description: "Names to ignore when running ShellCheck"
+    required: false
+    default: ""
   severity:
     description: "Minimum severity of errors to consider. Options: [error, warning, info, style]"
     required: false
@@ -26,10 +35,19 @@ inputs:
     description: "Set to true to skip using problem-matcher"
     required: false
     default: "false"
+    deprecationMessage: "There are no problem-matchers, this setting does not do anything."
+  format:
+    description: "Output format (checkstyle, diff, gcc, json, json1, quiet, tty)"
+    required: false
+    default: "gcc"
+  version:
+    description: "Specify a concrete version of ShellCheck to use"
+    required: false
+    default: "stable"
 outputs:
   files:
     description: A list of files with issues
-    value: ${{ steps.filepaths.outputs.filepaths }}
+    value: ${{ steps.check.outputs.filepaths }}
   options:
     description: The options used
     value: ${{ steps.options.outputs.options }}
@@ -38,16 +56,11 @@ branding:
   color: "gray-dark"
 runs:
   using: "composite"
-  steps: 
-    - name: Enable problem-matcher
-      shell: bash
-      run: |
-        if [[ ${{ inputs.disable_matcher }} != "true" ]]; then
-          echo "::add-matcher::${{ github.action_path }}/.github/problem-matcher.json"
-        fi
-
+  steps:
     - name: Download shellcheck
       shell: bash
+      env:
+        INPUT_VERSION: ${{ inputs.version }}
       run: |
         if [[ "${{ runner.os }}" == "macOS" ]]; then
           osvariant="darwin"
@@ -55,61 +68,102 @@ runs:
           osvariant="linux"
         fi
 
-        scversion="stable"
         baseurl="https://github.com/koalaman/shellcheck/releases/download"
 
         curl -Lso "${{ github.action_path }}/sc.tar.xz" \
-          "${baseurl}/${scversion}/shellcheck-${scversion}.${osvariant}.x86_64.tar.xz"
+          "${baseurl}/${INPUT_VERSION}/shellcheck-${INPUT_VERSION}.${osvariant}.x86_64.tar.xz"
 
         tar -xf "${{ github.action_path }}/sc.tar.xz" -C "${{ github.action_path }}"
-        mv "${{ github.action_path }}/shellcheck-${scversion}/shellcheck" \
+        mv "${{ github.action_path }}/shellcheck-${INPUT_VERSION}/shellcheck" \
           "${{ github.action_path }}/shellcheck"
 
+    - name: Display shellcheck version
+      shell: bash
+      run: |
+        "${{ github.action_path }}/shellcheck" --version
+
     - name: Set options
       shell: bash
       id: options
+      env:
+        INPUT_SEVERITY: ${{ inputs.severity }}
+        INPUT_FORMAT: ${{ inputs.format }}
       run: |
         declare -a options
-        if [[ -n "${{ inputs.severity }}" ]]; then
-          options+=("-S ${{ inputs.severity }}")
+        if [[ -n "${INPUT_SEVERITY}" ]]; then
+          options+=("-S ${INPUT_SEVERITY}")
         fi
-        echo "::set-output name=options::${options[@]}"
+        options+=("--format=${INPUT_FORMAT}")
+        echo "options=${options[@]}" >> $GITHUB_OUTPUT
 
     - name: Gather excluded paths
       shell: bash
       id: exclude
+      env:
+        INPUT_IGNORE: ${{ inputs.ignore }}
+        INPUT_IGNORE_PATHS: ${{ inputs.ignore_paths }}
+        INPUT_IGNORE_NAMES: ${{ inputs.ignore_names }}
       run: |
         declare -a excludes
-        excludes+=("! -path \"*./.git/*\"")
-        excludes+=("! -path \"*.go\"")
-        excludes+=("! -path \"*/mvnw\"")
-        for path in ${{ inputs.ignore }}; do
-          echo "::debug:: Adding "$path" to excludes"
-          excludes+=("! -path \"*./$path/*\"")
-          excludes+=("! -path \"*/$path/*\"")
+        set -f # temporarily disable globbing so that globs in input aren't expanded
+
+        excludes+=("! -path *./.git/*")
+        excludes+=("! -path *.go")
+        excludes+=("! -path */mvnw")
+        if [[ -n "${INPUT_IGNORE}" ]]; then
+          for path in ${INPUT_IGNORE}; do
+            excludes+=("! -path *./$path/*")
+            excludes+=("! -path */$path/*")
+            excludes+=("! -path $path")
+          done
+        else
+          for path in ${INPUT_IGNORE_PATHS}; do
+            excludes+=("! -path *./$path/*")
+            excludes+=("! -path */$path/*")
+            excludes+=("! -path $path")
+          done
+        fi
+
+        for name in ${INPUT_IGNORE_NAMES}; do
+          excludes+=("! -name $name")
         done
-        echo "::set-output name=excludes::${excludes[@]}"
+        echo "excludes=${excludes[@]}" >> $GITHUB_OUTPUT
+
+        set +f # re-enable globbing
 
     - name: Gather additional files
       shell: bash
       id: additional
+      env:
+        INPUT_ADDITIONAL_FILES: ${{ inputs.additional_files }}
       run: |
         declare -a files
-        for file in ${{ inputs.additional_files }}; do
-          echo "::debug:: Adding "$file" to excludes"
-          files+=("-o -name \"*$file\"")
+        for file in ${INPUT_ADDITIONAL_FILES}; do
+          files+=("-o -name *$file")
         done
-        echo "::set-output name=files::${files[@]}"
+        echo "files=${files[@]}" >> $GITHUB_OUTPUT
 
-    - name: Gather base file paths
+    - name: Run the check
       shell: bash
-      id: filepaths
+      id: check
+      env:
+        INPUT_SCANDIR: ${{ inputs.scandir }}
+        INPUT_CHECK_TOGETHER: ${{ inputs.check_together }}
+        INPUT_EXCLUDE_ARGS: ${{ steps.exclude.outputs.excludes }}
+        INPUT_ADDITIONAL_FILE_ARGS: ${{ steps.additional.outputs.files }}
+        INPUT_SHELLCHECK_OPTIONS: ${{ steps.options.outputs.options }}
       run: |
+        statuscode=0
         declare -a filepaths
-        shebangregex="^#! */[^ ]*/(env *)?[abkz]*sh"
+        shebangregex="^#! */[^ ]*/(env *)?[abk]*sh"
 
-        for path in $(find "${{ inputs.scandir }}" \
-            -type f -type f ${{ steps.exclude.outputs.excludes }} \
+        set -f # temporarily disable globbing so that globs in inputs aren't expanded
+
+        while IFS= read -r -d '' file; do
+          filepaths+=("$file")
+        done < <(find "${INPUT_SCANDIR}" \
+            ${INPUT_EXCLUDE_ARGS} \
+            -type f \
             '(' \
             -name '*.bash' \
             -o -name '.bashrc' \
@@ -137,63 +191,35 @@ runs:
             -o -path '*/.profile' \
             -o -path '*/profile' \
             -o -name '*.shlib' \
-            ${{ steps.additional.outputs.files }} \
-              ')'\
-            -print); do
-          filepaths+=("$path");
-        done
+            ${INPUT_ADDITIONAL_FILE_ARGS} \
+            ')' \
+            -print0)
 
-        for file in $(find "${{ inputs.scandir }}" ${{ steps.exclude.outputs.excludes }} -type f ! -name '*.*' -perm /111  -print); do
+        while IFS= read -r -d '' file; do
           head -n1 "$file" | grep -Eqs "$shebangregex" || continue
-          filepaths+=("$file");
-        done
-        echo "::set-output name=filepaths::${filepaths[@]}"
+          filepaths+=("$file")
+        done < <(find "${INPUT_SCANDIR}" \
+            ${INPUT_EXCLUDE_ARGS} \
+            -type f ! -name '*.*' -perm /111 \
+            -print0)
 
-    - name: Check bin subdirs
-      shell: bash
-      run: |
-        if  find "${{ inputs.scandir }}" ${{ steps.exclude.outputs.excludes }} -path '*bin/*/*' -type f -perm /111 -print |
-            grep .
-        then
-            echo "::warning:: subdirectories of bin directories are not usable via PATH"
-        fi
-
-    - name: Check no suffix in PATH
-      shell: bash
-      run: |
-        if  find "${{ inputs.scandir }}" ${{ steps.exclude.outputs.excludes }} -path '*bin/*' -name '*.*' -type f -perm /111 -perm /444 -print |
-            grep .
-        then
-            echo "::warning:: programs in PATH should not have a filename suffix"
-        fi
-
-    - name: Run the file check
-      id: check
-      shell: bash
-      run: |
-        statuscode=0
-
-        if [[ -n "${{ inputs.check_together }}" ]]; then
-           "${{ github.action_path }}/shellcheck" \
-           ${{ steps.options.outputs.options }} \
-           ${{ steps.filepaths.outputs.filepaths }} || statuscode=$?
+        if [[ -n "${INPUT_CHECK_TOGETHER}" ]]; then
+          "${{ github.action_path }}/shellcheck" \
+            ${INPUT_SHELLCHECK_OPTIONS} \
+            "${filepaths[@]}" || statuscode=$?
         else
-          for file in ${{ steps.filepaths.outputs.filepaths }}; do
-            echo "::debug::Checking $file"
+          for file in "${filepaths[@]}"; do
             "${{ github.action_path }}/shellcheck" \
-            ${{ steps.options.outputs.options }} \
-            "$file" || statuscode=$?;
+              ${INPUT_SHELLCHECK_OPTIONS} \
+              "$file" || statuscode=$?
           done
         fi
-        
-        echo "::set-output name=statuscode::$statuscode"
 
-    - name: Print information
+        echo "filepaths=${filepaths[@]}" >> $GITHUB_OUTPUT
+        echo "statuscode=$statuscode" >> $GITHUB_OUTPUT
+
+        set +f # re-enable globbing
+
+    - name: Exit action
       shell: bash
-      run: |
-        echo "Files: ${{steps.filepaths.outputs.filepaths}}"
-        echo "Excluded: ${{ steps.exclude.outputs.excludes }}"
-        echo "Options: ${{ steps.options.outputs.options }}"
-        echo "Status code: ${{steps.check.outputs.statuscode}}"
-
-        exit ${{steps.check.outputs.statuscode}}
+      run: exit ${{steps.check.outputs.statuscode}}

testfiles/duplicate_name.bash

@@ -0,0 +1,3 @@
+#!/bin/bash
+test="test"
+echo "$test"

testfiles/ignore/ignore.bash

@@ -0,0 +1,3 @@
+#!/bin/bash
+test="test"
+echo "$test"

testfiles/ignore/ignore.sh

@@ -1,5 +0,0 @@
-#!/bin/sh
-
-echo $test $test
-
-echo $test2

testfiles/ignore_single_file.sh

@@ -0,0 +1,4 @@
+#!/usr/bin/sh
+
+test="test"
+echo "$test"

testfiles/ignore_some/do_not_ignore.bash

@@ -0,0 +1,3 @@
+#!/bin/bash
+test="test"
+echo "$test"

testfiles/ignore_some/duplicate_name.bash

@@ -0,0 +1,3 @@
+#!/bin/bash
+test="test"
+echo "$test"

testfiles/ignore_some/ignore.bash

@@ -0,0 +1,3 @@
+#!/bin/bash
+test="test"
+echo "$test"

testfiles/scandir/run me.bash

@@ -0,0 +1,3 @@
+#!/usr/bin/env bash
+
+echo "I love spaces"