CakePHP3 Advent Calendar 2016 - Qiita
20日目が空いていたので。
CakePHP3のマイグレーションにはmigration_diffという機能があり、
差分から自動的にマイグレーションファイルを作ってくれる超便利な機能がある。
ただ1点、
afterオプションだけ自動的につけてくれない、
という仕様だったのがずっと気になっていた。
気になってソースを調べてみると、
なんとなく可能なんじゃないか、という感じに見えた。
でも付けられていないということは何か理由でもあるのだろうか?
と思いつつ対応してみたら簡単にafterをつけることができた。
せっかくなのでテストも作成し、issueでPRしていいか聞いてみたところOKが出たのでPR。
Milestoneにも入ったので問題なければ恐らくマージされるのではないかと思う。
Add 'after' attribute to migration_diff by dala00 · Pull Request #281 · cakephp/migrations · GitHub
ちなみにafterが無いのは特に理由はなくまだだれも対応していなかっただけだった。
migration_diffのテストは、実行後git statusしてみるとだいたい分かるが、
マイグレーションファイルが作成され、それを想定される出力結果ファイルと比較する、というもの。
MySQLやPostgresでファイルが分かれているのでそれぞれテストして出力結果ファイルを修正する。
そのあたりをきちっとやったら後は環境依存のエラーやよくわからないものが出るので、
forkしたリポジトリをTravis CIでチェックして本家と同じ感じになればOKだと思われる。
PRした時も同様のチェックが自動的に行われる。
せっかくのオープンソースなので気になったり不足を感じる点があったら是非PRしてみた方が良い。
あとはマージされることを祈るばかり。
ちなみに仕事場ではもう勝手に修正をあてている。
もしマージされなかったら永久に自分で修正し続けるしかない。
追記)マージされました。
Crieitは誰でも投稿できるサービスです。 是非記事の投稿をお願いします。どんな軽い内容でも投稿できます。
また、「こんな記事が読みたいけど見つからない!」という方は是非記事投稿リクエストボードへ!
こじんまりと作業ログやメモ、進捗を書き残しておきたい方はボード機能をご利用ください。
ボードとは?
コメント