phpcsとphpmdを利用した継続的インテグレーション

Transcript

1. phpcsとphpmdを利用した 継続的インテグレーション @マッチングッドさいとう
2. 自己紹介 マッチングッド株式会社について 人材会社向けのクラウドの基幹システムを運営している会社 toBでPHPを利用しておりモダンな技術で顧客価値の向上を目指し中 私について マッチングッドで代表を務めさせていただいております 技術的にはソースレビューの8割を行っています
3. 今日の概要 当社で行っているphpcsとphpmdを利用した 継続的インテグレーションについてご紹介 継続的インテグレーションとは？ 継続的インテグレーション、CI（英: continuous integration）と は、主にプログラマーのアプリケーション作成時の品質改善や納期の短縮 のための習慣 @wikipedia
4. 抱えていた課題 コーディング規約（PSRにほぼ準拠）があるのに、 なぜかコーディング規約が遵守されないソースがpushされる・・・ 皆さんならPSRを守ってもらうためにどうしますか？
5. 最初にやったこと .editorconfigの準備 エディタの設定で何とかしようとしたが・・・ [*] charset = utf‐8 indent_size = 4 indent_style = space trim_trailing_whitespace = true insert_final_newline = true phpcsとphpmdをコマンドで叩く phpcs：コーディング規約を遵守させるためのツール phpmd：潜在バグをチェックするためのツール ルールは作ったが・・・
6. 別の問題 .editorconfigではコーディング規約の全てをカバーできない コーディング規約の中にある未使用の変数を使わないなどの 潜在バグをキャッチできない・・・ phpcsとphpmdをコマンドで叩くのを忘れる
7. だって忘れるよね どうしたら忘れないようにできるでしょうか？
8. 解決策 当社ではソース管理をgithubで行ってますが、 CircleCIと連携させました。 そして、CircleCIでphpcsとphpmdをソースのpush時に動かし、 ユニットテストだけでなく、phpcsとphpmdもパスしないと、 マスターにマージできないようにしました。 circle.ymlの一部 vendor/bin/phpcs ‐‐standard=phpcs.xml vendor/bin/phpunit vendor/bin/phpmd app text phpmd.xml
9. イメージ githubのイメージ circleCIのイメージ
10. 結果 コーディング規約に関するレビュー量が8割減！！
11. We are hiring! ご清聴ありがとうございました！