去年の年末に大掃除も年賀状もやらずに Rails に Pull Request を出してた。
年末だけど大掃除もせず年賀状も書かず実家にも帰省せず rails/rails に Pull Request 出した。https://t.co/a0xePd51vY
— morygonzalez (@morygonzalez) December 30, 2017
ActionMailer のプレビューで locale が複数ある場合に指定できるようにするというもの。 Kaizen Platform の Rails アプリにはこの機能付いてて多言語対応のメールをプレビューするときにめっちゃ便利だった。調べたところ Rails 4 時代にそういう Pull Request 出してた人がいて Merge 寸前まで行ってたんだけど commit が複数に分かれてたのを「 squash してくれない?」とレビューされたところでプルリク主の意欲が燃え尽きたっぽくて Merge されずにコンフリクトして死んでた。
Rails 5 でも動くようにコンフリクトを解消してテストケースも追加したのが以下。
動作イメージはこんな感じ。
ただ Merge 後にバグってるのを指摘されていま直してるところです。
頭良くないのでこういうしょぼい Pull Request でしか contribute できないけど自分にできる範囲で貢献していきたい。
追記 2018-01-24
問題を修正する Pull Request も Merge してもらったんで多分 Rails 5.2 にこの機能入ります。