なるようになるブログ

読書感想文かrailsについてかrubyについてか

rails commit log流し読み(2018/05/24)

2018/05/24分のコミットです。

CHANGELOGにのったコミットは以下の通りです。

actionmailer/CHANGELOG.md


Merge pull request #32882 from utilum/unnecessary_mocking

Action Cableのテストの修正です。

mockしないで良い箇所までmockを使ってしまっていたのを修正しています。


Change video preview format from PNG to JPG

activestorage/lib/active_storage/previewer/video_previewer.rbの修正です。

video previewで表示する画像をPNGからJPGに変更しています。

JPGの方がファイルサイズが小さい為。


Eager autoload mail gem when eager load is true (#32808)

actionmailer/lib/action_mailer.rbの修正です。

eager loadがtrueの場合、mail gemをeager autoloadするよう修正しています。

元々はaction_mailer/base.rbで普通にrequireしていたのですが、それだとmail gemの読み込み時にSidekiq worker threadがdeadlockしてしまうケースがあった為、eager loadするよう修正しています。


Enable warnings in all test tasks

テスト実行時にRubyのワーニングが表示されるよう、全てのコンポーネントRakefilewarningオプションにtrueを指定しています。


Merge pull request #30406 from eliotsykes/reduce-spring-watch-calls

railties/lib/rails/generators/rails/app/templates/config/spring.rb.ttの修正です。

Spring.watchに監視したいファイルを渡すのに、ファイル毎にSpring.watchメッセージを呼び出していたのを、Spring.watchにファイルを配列で渡すようにしています。Spring.watchメソッドの呼び出しを減らせるようにする為。


Avoid 2.6 warning: shadowing outer local variable - list

activestorage/app/models/active_storage/variation.rbの修正です。

ActiveStorage::Variation#image_processing_transformメソッドでshadowing outer local variable - listのwarningが出ていたのを修正しています。


correct link to activejob readme [ci skip]

activejob/lib/active_job/queue_adapters.rbのdocの修正です。

ActiveJob::QueueAdapters moduleのdoc内、READMEについて言及している箇所で、実際のファイルへのリンクを追加しています。