コミットされるlaravelのコードだけをフォーマットする
以前の記事でPHPをフォーマットができるようになった。
ただこれを実行すると既存のコードをすべて書き換えてしまうので、チームの合意が得づらい。 なのでコミットされるファイルだけをフォーマットしていき、徐々にフォーマットされている世界を目指す事にした。
設定サンプルリポジトリ
この記事の設定を適用したサンプル
使用技術
phpでどうやるのかがわからないが、laravelにはpackage.json
つまりはNode.jsの技術が入っています。なので基本的にはNode.jsの技術に全部頼ることにする。
というわけでhuskyとlint-sagedを使う。
導入
インストールや初期設定。
基本的には設定者以外の人はgit clone
してきて、composer install
, npm install
すれば勝手に動く状態を目指す。
phpのフォーマットは以前の記事でも使用したphp-cs-fixerを使用します。ルール設定等も以前の記事を参考にしてください。
また設定にはnpmを使用するという前提で書きます。もしyarnなどを使ってる場合は適宜置き換えてください。
npm i -D husky lint-staged # huskyの初期準備 npm set-script prepare "husky install" npm run prepare # commit hookの設定 npx husky add .husky/pre-commit "npx lint-staged" # lint-stagedの設定ファイルを作る touch .lintstagedrc.json # package.jsonにフォーマット用の設定を作成 # php-cs-fixerのコンフィグファイルを指定する npm set-script format './vendor/bin/php-cs-fixer fix --config .php-cs-fixer.dist.php'
lint-stagedの設定
先程作成した.lintstagedrc.json
に設定を記述していきます。
設定自体はphp-cs-fixerの以前の記事*1を参考に記述します
{ "./app/**/*.php": [ "npm run format" ], "./config/**/*.php": [ "npm run format" ], "./database/factories/**/*.php": [ "npm run format" ], "./database/seeders/**/*.php": [ "npm run format" ], "./routes/**/*.php": [ "npm run format" ], "./tests/**/*.php": [ "npm run format" ], "./.php-cs-fixer.dist.php": [ "npm run format" ] }
これでlint-stagedの設定で指定したファイルが変更された時に自動的にphp-cs-fixerが実行されるようになる。
ちなみに
bladeのフォーマットにはblade-formatterというのがあるらしいのでそれを設定してやれば動くはず。
インストールして雑にpackage.json
のscriptsに登録する。
npm i -D blade-formatter npm set-script format:blade 'blade-formatter -w'
あとは.lintstagedrc.json
にblade用の設定を追加
{ "./resources/**/*.blade.php": ["npm run format:blade"] }
php-cs-fixerで設定ファイルをフォーマットする
最近仕事でPHP × Laravelを書くことを迫られてて、普段phpの情報は追ってないので環境を整えることから始めている。
とりあえずフォーマット頑張るのアホらしいのでjavascriptでいうpretteir的なものを求めて探しているとphp-cs-fixerというものに出会った。
結論だけ知りたい方は最後まで飛ばしてください
導入
公式見てもよくわからなかったのでこちらのqiita記事を参考にして導入した。 記事を参考にコマンド等ポチポチしてるとハマることなくインストールできた。
本題
参考にした記事でほとんどのファイルをフォーマットできたが設定ファイルの.php-cs-fixer.dist.php
がフォーマットされないな〜と思って
in
の部分の記述を参考に以下のものを記述したがうまく動かずappend
メソッドを見つけたのでそっちでやっても動かなかった。
<?php // inメソッドはディレクトリを指定するものでファイルの指定はできないっぽい $finder = PhpCsFixer\Finder::create() ->in([ // 省略 __DIR__.'/.php-cs-fixer.dist.php' ]) // これも動かない $finder = PhpCsFixer\Finder::create() ->in([ // 省略 ]) ->append([ __DIR__.'/.php-cs-fixer.dist.php' ])
解決策
仕方がないのでリポジトリを見に行くと、設定ファイルが置いてあり、そこに理由と解決策が書いていた*1
__DIR__.'/php-cs-fixer'
, disabled, as we want to be able to run bootstrap file even on lower PHP version, to show nice message
らしく、以下のような指定でフォーマットすることができた
<?php $finder = PhpCsFixer\Finder::create() ->in([ // 省略 ]) ->append([ __FILE__ ])
最終的に
ルールがよくわからんのでphp-cs-fixerが用意してるっぽいのとimportのorderだけ並び変える設定を入れた
<?php declare(strict_types=1); $finder = PhpCsFixer\Finder::create() ->in([ __DIR__.'/app', __DIR__.'/config', __DIR__.'/database/factories', __DIR__.'/database/seeders', __DIR__.'/routes', __DIR__.'/tests', ]) ->append([ __FILE__, ]) ; $config = new PhpCsFixer\Config(); return $config ->setRiskyAllowed(true) ->setRules([ '@PhpCsFixer' => true, '@PhpCsFixer:risky' => true, 'ordered_imports' => ['sort_algorithm' => 'alpha', 'imports_order' => ['const', 'class', 'function']], ]) ->setFinder($finder) ;
node-cronのv3でハマった
そんな事あるのかって思ったのでメモ
tl;dr
node-cronのtask.destory()
がなくなっていた
概要
家のラズパイで動いてるプログラムの依存をアップデートしたらnode-cron周りでエラーを吐くようになってしまった。エラーを見るとtask.destory()
が呼べなかったらしい。
問題の解決
node-cronは特にリリースノートとかないっぽい?のでtask.destory()
がなくなっていたことに気づかなかった。
自分が調べた時にはまだREADME.md
にはdestory()
が残っていて無くなったらならバージョン上げたやつリリースする前に削除しててほしい…
というわけでtask.destory()
をtask.stop()
に変えるとcronを止める事ができました。