fix: exclude option doesn't work (#82)

reviewdog · Jan 19, 2023 · 141ffd8 · 141ffd8
1 parent 1b07070
commit 141ffd8
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 1 deletion.
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -15,6 +15,7 @@ jobs:
         with:
           reporter: github-check
           level: info
+          exclude: Dockerfile.dockerignore
 
   test-pr-review:
     if: github.event_name == 'pull_request'
@@ -26,3 +27,4 @@ jobs:
         with:
           reporter: github-pr-review
           filter_mode: diff_context
+          exclude: Dockerfile.dockerignore
diff --git a/script.sh b/script.sh
@@ -20,7 +20,8 @@ export REVIEWDOG_GITHUB_API_TOKEN="${INPUT_GITHUB_TOKEN}"
 
 EXCLUDES=""
 for exclude_path in $INPUT_EXCLUDE; do
-  EXCLUDES="$EXCLUDES --exclude='!$exclude_path'"
+  set -- --exclude="!${exclude_path}"
+  EXCLUDES="$EXCLUDES $*"
 done
 
 IGNORE_LIST=""

diff --git a/testdata/Dockerfile.dockerignore b/testdata/Dockerfile.dockerignore
@@ -0,0 +1 @@
+.git