reviewdog で構文チェックや静的解析の結果をプルリクのレビューコメントとして出力してみます。プルリクエストを作成した時に構文チェックや静的解析の検査結果をプルリクのレビューコメントに出力してあげると色々と手間も減って少しだけ幸せになれます。

reviewdog で構文チェックや静的解析の結果をプルリクのレビューコメントとして出力する（GithubActions）

プルリクエストを作成した時に構文チェックや静的解析を走らせてソースコードのチェックを行っていた際に、検査に通らなかった場合は結果エラーと表示・通知されますが、その度に GithubActions / CircleCI / TravisCI などの結果画面へ遷移して検査に通らなかった箇所を確認するのって少しだけ面倒だったりします。
そんな時は、reviewdog を使って、検査結果をプルリクのレビューコメントに出力してあげると手間も減って少しだけ幸せになれます。
今回は、reviewdog で構文チェックや静的解析の結果をプルリクのレビューコメントとして出力してみます。
<h2 id="contents">Contents</h2>
<ol>
<li><a href="#reviewdog">reviewdog</a></li>
<li><a href="#%E6%A4%9C%E8%A8%BC%E7%92%B0%E5%A2%83">検証環境</a></li>
<li><a href="#%E6%A7%8B%E6%96%87%E3%83%81%E3%82%A7%E3%83%83%E3%82%AF%E3%81%AE%E3%82%A8%E3%83%A9%E3%83%BC%E3%82%92%E3%82%B3%E3%83%A1%E3%83%B3%E3%83%88%E3%81%AB%E5%87%BA%E5%8A%9B%E3%81%99%E3%82%8B">構文チェックのエラーをコメントに出力する</a></li>
<li><a href="#%E9%9D%99%E7%9A%84%E8%A7%A3%E6%9E%90%E3%81%AE%E3%82%A8%E3%83%A9%E3%83%BC%E3%82%92%E3%82%B3%E3%83%A1%E3%83%B3%E3%83%88%E3%81%AB%E5%87%BA%E5%8A%9B%E3%81%99%E3%82%8B">静的解析のエラーをコメントに出力する</a>
<ol>
<li><a href="#%E8%A3%9C%E8%B6%B3">補足</a></li>
</ol>
</li>
</ol>
<h2 id="reviewdog">reviewdog</h2>
reviewdog は、レビューコメントを出力してくれるツールです。 
<a href="https://github.com/reviewdog/reviewdog" rel="noopener noreferrer" target="_blank">https://github.com/reviewdog/reviewdog</a>
<h2 id="検証環境">検証環境</h2>
今回使用するツールは以下の通りです
<ul>
<li>reviewdog</li>
<li>Github Actions</li>
<li>PHP_CodeSniffer（構文チェック）</li>
<li>PHPStan（静的解析）</li>
</ul>
今回は Github Actions に reviewdog を導入して、プルリクエストで構文チェックと静的解析を走らせ、検査に通らないものがあった場合にその結果をレビューコメントとして出力したいと思います。
一部 PHP フレームワーク Laravel のセットアップが出てきますが、特に Laravel のコードは出てきません。プロジェクトルートで各検査を実行している事を明示するために残しているだけなので、他の FW やスクラッチでも設定としては問題ありません。
<h2 id="構文チェックのエラーをコメントに出力する">構文チェックのエラーをコメントに出力する</h2>
まずは、PHP_CodeSniffer にて構文チェックを行い、エラーがあればプルリクのコメントに出力するようにします。
reviewdog なしで通常の構文チェックを走らせる場合の Action は以下になっています。（以下すべて yml は該当部分のみ抜粋して表示しています）
<dl class="description_setting">
<dt>.github/workflows/ci.yml</dt>
<dd>
<pre><code class="hljs language-yaml">jobs:
 # 構文チェック
 lint:
 runs-on: ubuntu-latest
 steps:
 - uses: actions/checkout@v2
 - uses: rito328/composite-run-steps-action-laravel-setup@v1 # Laravel setup
 - name: lint
 run: ./vendor/bin/phpcs --report=emacs --standard=phpcs.xml ./
</code></pre>
</dd>
</dl>
（Laravel setup のところは、個人的に Laravel のセットアップをまとめている Composite Action なので、試す場合は削除して各々の定義を追加してください）
ここに reviewdog を導入してレビューコメントを出力するようにするので、以下のように変更します。
<dl class="description_setting">
<dt>.github/workflows/ci.yml</dt>
<dd>
<pre><code class="hljs language-yaml">jobs:
 # 構文チェック
 lint:
 runs-on: ubuntu-latest
 steps:
 - uses: actions/checkout@v2
 - uses: rito328/composite-run-steps-action-laravel-setup@v1
 # 1. reviewdog の setup action を追加
 - uses: reviewdog/action-setup@v1
 with:
 reviewdog_version: latest
 - name: lint
 env:
 # 2. reviewdog が コメントを書き込めるように token をセットする
 REVIEWDOG_GITHUB_API_TOKEN: ${{ secrets.GITHUB_TOKEN }}
 # 3. 構文チェックの結果を reviewdog へ渡す
 run: ./vendor/bin/phpcs --report=emacs --standard=phpcs.xml ./ | reviewdog -reporter=github-pr-review -efm='%f:%l:%c:%m'
</code></pre>
</dd>
</dl>
<ol>
<li>reviewdog の setup action を追加して reviewdog を導入しています</li>
<li>reviewdog が コメントを書き込めるように GITHUB_TOKEN をセットしています</li>
<li>構文チェックの結果を reviewdog へ渡しています</li>
</ol>
ポイントとしては、コメントを書き込める用に環境変数 REVIEWDOG_GITHUB_API_TOKEN に GITHUB_TOKEN 渡してあげることがまず一つです。ただ結果を渡すだけでは、書き込みの権限がなくて失敗してしまいます。
２つ目は、lint 結果のフォーマットです。
通常のエラーレポートの出力フォーマットでは reviewdog 側で簡単に読み取ることができないので、--report=emacs を指定して、出力フォーマットを通常のものから変更しています。
その上で、コメントのフォーマットを reviewdog 側で指定しています。（-efm='%f:%l:%c:%m'）
Reporting 
<a href="https://github.com/squizlabs/PHP_CodeSniffer/wiki/Reporting" rel="noopener noreferrer" target="_blank">https://github.com/squizlabs/PHP_CodeSniffer/wiki/Reporting</a>
これで、構文チェックに引っかかったものが、プルリクのレビューコメントに出力されるようになります。
<div class="img_wrap"><amp-img layout="responsive" src="/images/posts/213/pRb1GqwZB7D0DIsgIXqm.png" width="800" height="378" alt="構文チェックエラーのレビューコメント"></div>
<h2 id="静的解析のエラーをコメントに出力する">静的解析のエラーをコメントに出力する</h2>
次は、PHPStan にて静的解析を行い、エラーがあればプルリクのコメントに出力するようにします。
まず、reviewdog なしで通常の構文チェックを走らせる場合の Action は以下になっています。
<dl class="description_setting">
<dt>.github/workflows/ci.yml</dt>
<dd>
<pre><code class="hljs language-yaml">jobs:
 analyse:
 runs-on: ubuntu-latest
 steps:
 - uses: actions/checkout@v2
 - uses: rito328/composite-run-steps-action-laravel-setup@v1
 - name: analyse
 run: ./vendor/bin/phpstan analyse
</code></pre>
</dd>
</dl>
各々セットアップして、PHPStan を実行しているだけの、ベーシックな定義です。
これに、reviewdog を導入するために以下の記述に変更します。
<dl class="description_setting">
<dt>.github/workflows/ci.yml</dt>
<dd>
<pre><code class="hljs language-yaml">jobs:
 # 静的解析
 analyse:
 runs-on: ubuntu-latest
 steps:
 - uses: actions/checkout@v2
 - uses: rito328/composite-run-steps-action-laravel-setup@v1
 # 1. reviewdog の setup action を追加
 - uses: reviewdog/action-setup@v1
 with:
 reviewdog_version: latest
 - name: analyse
 env:
 # 2. reviewdog が コメントを書き込めるように token をセットする
 REVIEWDOG_GITHUB_API_TOKEN: ${{ secrets.GITHUB_TOKEN }}
 # 3. 静的解析の結果を reviewdog へ渡す
 run: ./vendor/bin/phpstan analyse --error-format=raw --no-progress | reviewdog -reporter=github-pr-review -f=phpstan
</code></pre>
</dd>
</dl>
基本的な部分は構文チェックの時と変わりませんが、一つだけポイントがあります。
<pre><code class="hljs language-yaml">run: ./vendor/bin/phpstan analyse --error-format=raw --no-progress | reviewdog -reporter=github-pr-review -f=phpstan
</code></pre>
reviewdog は PHPStan 用のメッセージフォーマットを既に持っているので、-f=phpstan とするだけで済むのですが、今回の書き方の場合、PHPStan 側のエラー出力がデフォルトでは読み取れないので、--error-format=raw を指定してエラーの出力フォーマットを変更してあげる必要があります。
PHPStan - Output Format 
<a href="https://phpstan.org/user-guide/output-format" rel="noopener noreferrer" target="_blank">https://phpstan.org/user-guide/output-format</a>
これで、静的解析に引っかかったものが、プルリクのレビューコメントに出力されるようになります。
<div class="img_wrap"><amp-img layout="responsive" src="/images/posts/213/D22ZqPm9RskdHY9Zc2go.png" width="800" height="296" alt="静的解析エラーのレビューコメント"></div>
<h3 id="補足">補足</h3>
PHP_CodeSniffer と PHPStan のエラー出力のフォーマットをデフォルトから変更しましたが、基本的デフォルトでは検査結果は人が確認しやすいような書式で出力されるので、シンプルな出力に変更する事で reviewdog から読み取れるようにしています。
ただ、reviewdog 自体は checkstyle を読めるので、今回のように出力フォーマットを変更しなくても設定次第でいけるのかもというのは一応補足しておきます。（検証はしていません）
（とはいいつつ、-f=phpstan で使う場合、PHPStan の場合は raw フォーマット前提のようです） 
<a href="https://github.com/reviewdog/errorformat/pull/22/files" rel="noopener noreferrer" target="_blank">https://github.com/reviewdog/errorformat/pull/22/files</a>
<h2 id="まとめ">まとめ</h2>
reviewdog を導入するとそのプルリクを出した際の検査結果がコメントが書き込まれるので、おそらく最も早くレビューのコメントがつく。という事を考えると、人が実際にレビューを行う前に基本的な部分が自動で指摘されるので、全体的なレビュー労力の削減にもなりそうですね。
本当はこういった類のものはプルリク出す前に全て潰してしまえたら最良ですが、漏れる時もありますし、reviewdog にかわいく（コメントに犬のアイコンが出てきます）指摘してもらうのも良いかもしれません。

プルリクエストを作成した時に構文チェックや静的解析を走らせてソースコードのチェックを行っていた際に、検査に通らなかった場合は結果エラーと表示・通知されますが、その度に GithubActions / CircleCI / TravisCI などの結果画面へ遷移して検査に通らなかった箇所を確認するのって少しだけ面倒だったりします。
そんな時は、reviewdog を使って、検査結果をプルリクのレビューコメントに出力してあげると手間も減って少しだけ幸せになれます。
今回は、reviewdog で構文チェックや静的解析の結果をプルリクのレビューコメントとして出力してみます。
<h2 id="contents">Contents</h2>
<ol>
<li><a href="#reviewdog">reviewdog</a></li>
<li><a href="#%E6%A4%9C%E8%A8%BC%E7%92%B0%E5%A2%83">検証環境</a></li>
<li><a href="#%E6%A7%8B%E6%96%87%E3%83%81%E3%82%A7%E3%83%83%E3%82%AF%E3%81%AE%E3%82%A8%E3%83%A9%E3%83%BC%E3%82%92%E3%82%B3%E3%83%A1%E3%83%B3%E3%83%88%E3%81%AB%E5%87%BA%E5%8A%9B%E3%81%99%E3%82%8B">構文チェックのエラーをコメントに出力する</a></li>
<li><a href="#%E9%9D%99%E7%9A%84%E8%A7%A3%E6%9E%90%E3%81%AE%E3%82%A8%E3%83%A9%E3%83%BC%E3%82%92%E3%82%B3%E3%83%A1%E3%83%B3%E3%83%88%E3%81%AB%E5%87%BA%E5%8A%9B%E3%81%99%E3%82%8B">静的解析のエラーをコメントに出力する</a>
<ol>
<li><a href="#%E8%A3%9C%E8%B6%B3">補足</a></li>
</ol>
</li>
</ol>
<h2 id="reviewdog">reviewdog</h2>
reviewdog は、レビューコメントを出力してくれるツールです。 
<a href="https://github.com/reviewdog/reviewdog" rel="noopener noreferrer" target="_blank">https://github.com/reviewdog/reviewdog</a>
<h2 id="検証環境">検証環境</h2>
今回使用するツールは以下の通りです
<ul>
<li>reviewdog</li>
<li>Github Actions</li>
<li>PHP_CodeSniffer（構文チェック）</li>
<li>PHPStan（静的解析）</li>
</ul>
今回は Github Actions に reviewdog を導入して、プルリクエストで構文チェックと静的解析を走らせ、検査に通らないものがあった場合にその結果をレビューコメントとして出力したいと思います。
一部 PHP フレームワーク Laravel のセットアップが出てきますが、特に Laravel のコードは出てきません。プロジェクトルートで各検査を実行している事を明示するために残しているだけなので、他の FW やスクラッチでも設定としては問題ありません。
<h2 id="構文チェックのエラーをコメントに出力する">構文チェックのエラーをコメントに出力する</h2>
まずは、PHP_CodeSniffer にて構文チェックを行い、エラーがあればプルリクのコメントに出力するようにします。
reviewdog なしで通常の構文チェックを走らせる場合の Action は以下になっています。（以下すべて yml は該当部分のみ抜粋して表示しています）
<dl class="description_setting">
<dt>.github/workflows/ci.yml</dt>
<dd>
<pre><code class="hljs language-yaml">jobs:
 # 構文チェック
 lint:
 runs-on: ubuntu-latest
 steps:
 - uses: actions/checkout@v2
 - uses: rito328/composite-run-steps-action-laravel-setup@v1 # Laravel setup
 - name: lint
 run: ./vendor/bin/phpcs --report=emacs --standard=phpcs.xml ./
</code></pre>
</dd>
</dl>
（Laravel setup のところは、個人的に Laravel のセットアップをまとめている Composite Action なので、試す場合は削除して各々の定義を追加してください）
ここに reviewdog を導入してレビューコメントを出力するようにするので、以下のように変更します。
<dl class="description_setting">
<dt>.github/workflows/ci.yml</dt>
<dd>
<pre><code class="hljs language-yaml">jobs:
 # 構文チェック
 lint:
 runs-on: ubuntu-latest
 steps:
 - uses: actions/checkout@v2
 - uses: rito328/composite-run-steps-action-laravel-setup@v1
 # 1. reviewdog の setup action を追加
 - uses: reviewdog/action-setup@v1
 with:
 reviewdog_version: latest
 - name: lint
 env:
 # 2. reviewdog が コメントを書き込めるように token をセットする
 REVIEWDOG_GITHUB_API_TOKEN: ${{ secrets.GITHUB_TOKEN }}
 # 3. 構文チェックの結果を reviewdog へ渡す
 run: ./vendor/bin/phpcs --report=emacs --standard=phpcs.xml ./ | reviewdog -reporter=github-pr-review -efm='%f:%l:%c:%m'
</code></pre>
</dd>
</dl>
<ol>
<li>reviewdog の setup action を追加して reviewdog を導入しています</li>
<li>reviewdog が コメントを書き込めるように GITHUB_TOKEN をセットしています</li>
<li>構文チェックの結果を reviewdog へ渡しています</li>
</ol>
ポイントとしては、コメントを書き込める用に環境変数 REVIEWDOG_GITHUB_API_TOKEN に GITHUB_TOKEN 渡してあげることがまず一つです。ただ結果を渡すだけでは、書き込みの権限がなくて失敗してしまいます。
２つ目は、lint 結果のフォーマットです。
通常のエラーレポートの出力フォーマットでは reviewdog 側で簡単に読み取ることができないので、--report=emacs を指定して、出力フォーマットを通常のものから変更しています。
その上で、コメントのフォーマットを reviewdog 側で指定しています。（-efm='%f:%l:%c:%m'）
Reporting 
<a href="https://github.com/squizlabs/PHP_CodeSniffer/wiki/Reporting" rel="noopener noreferrer" target="_blank">https://github.com/squizlabs/PHP_CodeSniffer/wiki/Reporting</a>
これで、構文チェックに引っかかったものが、プルリクのレビューコメントに出力されるようになります。
<div class="img_wrap"><img src="/images/posts/213/pRb1GqwZB7D0DIsgIXqm.png" width="800" height="378" alt="構文チェックエラーのレビューコメント"></div>
<h2 id="静的解析のエラーをコメントに出力する">静的解析のエラーをコメントに出力する</h2>
次は、PHPStan にて静的解析を行い、エラーがあればプルリクのコメントに出力するようにします。
まず、reviewdog なしで通常の構文チェックを走らせる場合の Action は以下になっています。
<dl class="description_setting">
<dt>.github/workflows/ci.yml</dt>
<dd>
<pre><code class="hljs language-yaml">jobs:
 analyse:
 runs-on: ubuntu-latest
 steps:
 - uses: actions/checkout@v2
 - uses: rito328/composite-run-steps-action-laravel-setup@v1
 - name: analyse
 run: ./vendor/bin/phpstan analyse
</code></pre>
</dd>
</dl>
各々セットアップして、PHPStan を実行しているだけの、ベーシックな定義です。
これに、reviewdog を導入するために以下の記述に変更します。
<dl class="description_setting">
<dt>.github/workflows/ci.yml</dt>
<dd>
<pre><code class="hljs language-yaml">jobs:
 # 静的解析
 analyse:
 runs-on: ubuntu-latest
 steps:
 - uses: actions/checkout@v2
 - uses: rito328/composite-run-steps-action-laravel-setup@v1
 # 1. reviewdog の setup action を追加
 - uses: reviewdog/action-setup@v1
 with:
 reviewdog_version: latest
 - name: analyse
 env:
 # 2. reviewdog が コメントを書き込めるように token をセットする
 REVIEWDOG_GITHUB_API_TOKEN: ${{ secrets.GITHUB_TOKEN }}
 # 3. 静的解析の結果を reviewdog へ渡す
 run: ./vendor/bin/phpstan analyse --error-format=raw --no-progress | reviewdog -reporter=github-pr-review -f=phpstan
</code></pre>
</dd>
</dl>
基本的な部分は構文チェックの時と変わりませんが、一つだけポイントがあります。
<pre><code class="hljs language-yaml">run: ./vendor/bin/phpstan analyse --error-format=raw --no-progress | reviewdog -reporter=github-pr-review -f=phpstan
</code></pre>
reviewdog は PHPStan 用のメッセージフォーマットを既に持っているので、-f=phpstan とするだけで済むのですが、今回の書き方の場合、PHPStan 側のエラー出力がデフォルトでは読み取れないので、--error-format=raw を指定してエラーの出力フォーマットを変更してあげる必要があります。
PHPStan - Output Format 
<a href="https://phpstan.org/user-guide/output-format" rel="noopener noreferrer" target="_blank">https://phpstan.org/user-guide/output-format</a>
これで、静的解析に引っかかったものが、プルリクのレビューコメントに出力されるようになります。
<div class="img_wrap"><img src="/images/posts/213/D22ZqPm9RskdHY9Zc2go.png" width="800" height="296" alt="静的解析エラーのレビューコメント"></div>
<h3 id="補足">補足</h3>
PHP_CodeSniffer と PHPStan のエラー出力のフォーマットをデフォルトから変更しましたが、基本的デフォルトでは検査結果は人が確認しやすいような書式で出力されるので、シンプルな出力に変更する事で reviewdog から読み取れるようにしています。
ただ、reviewdog 自体は checkstyle を読めるので、今回のように出力フォーマットを変更しなくても設定次第でいけるのかもというのは一応補足しておきます。（検証はしていません）
（とはいいつつ、-f=phpstan で使う場合、PHPStan の場合は raw フォーマット前提のようです） 
<a href="https://github.com/reviewdog/errorformat/pull/22/files" rel="noopener noreferrer" target="_blank">https://github.com/reviewdog/errorformat/pull/22/files</a>
<h2 id="まとめ">まとめ</h2>
reviewdog を導入するとそのプルリクを出した際の検査結果がコメントが書き込まれるので、おそらく最も早くレビューのコメントがつく。という事を考えると、人が実際にレビューを行う前に基本的な部分が自動で指摘されるので、全体的なレビュー労力の削減にもなりそうですね。
本当はこういった類のものはプルリク出す前に全て潰してしまえたら最良ですが、漏れる時もありますし、reviewdog にかわいく（コメントに犬のアイコンが出てきます）指摘してもらうのも良いかもしれません。

Author