Skip to content

Commit 78355bb

Browse files
Stephan202rickie
authored andcommitted
Improve Reviewdog issue reporting
A change in one place (such as a dependency upgrade or the introduction of a new Refaster rule) can trigger a build warning in some other place. With this change, Reviewdog also reports warnings against unaltered files. While there, drop the redundant `-tee` flag.
1 parent b9c51bf commit 78355bb

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

.github/workflows/reviewdog.yml

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -59,9 +59,8 @@ jobs:
5959
| reviewdog \
6060
-name=javac \
6161
-efm='[WARNING] %f:[%l,%c] %m' \
62-
-filter-mode=file \
63-
-reporter=github-pr-review \
64-
-tee
62+
-filter-mode=nofilter \
63+
-reporter=github-pr-review
6564
find -name checkstyle-result.json -exec sh -c '
6665
reviewdog -name=Checkstyle -f=sarif -reporter=github-pr-review < "{}"
6766
' \;

0 commit comments

Comments
 (0)