2018-08-15に更新

cakephp/migrationsにプルリクエストを送った

はじめに

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開発者

Crieitの開発者です。 Webエンジニアです(在宅)。大体10年ちょい。 記事でわかりにくいところがあればDMで質問していただくか、案件発注してください。 業務依頼、同業種の方からのコンタクトなどお気軽にご連絡ください。 業務経験有:PHP, MySQL, Laravel, React, Flutter, Vue.js, Node, RoR 趣味:Elixir, Phoenix, Nuxt, Express, GCP, AWS等色々 PHPフレームワークちいたんの作者

Crieitは誰でも投稿できるサービスです。 是非記事の投稿をお願いします。どんな軽い内容でも投稿できます。

また、「こんな記事が読みたいけど見つからない!」という方は是非記事投稿リクエストボードへ!

有料記事を販売できるようになりました!

こじんまりと作業ログやメモ、進捗を書き残しておきたい方はボード機能をご利用ください。
ボードとは?

コメント