なるようになるブログ

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

rails commit log流し読み(2019/04/17)

2019/04/17分のコミットです。

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

activesupport/CHANGELOG.md

activerecord/CHANGELOG.md


Update the generate command sample codes [skip ci]

rails guideのThe Rails Command Lineの修正です。

rails generateの項にあるコマンド実行結果のexampleを、実際のコマンドの実行結果に合わせて修正しています。


Squash warning: possibly useless use of a constant

railties/test/application/configuration_test.rbの修正です。

possibly useless use of a constantのwarningが出ていたのを修正しています。


[Rails::MailersController] Do not leak I18n global setting changes

railties/lib/rails/mailers_controller.rbの修正です。

mailer preivewでlocaleを変更した際に、globalの状態(I18n.locale)を変更してしまっていたのを、I18n.with_localeを使用してglobalの状態は変更しないよう修正しています。


running test with_info_handler method

activesupport/lib/active_support/testing/parallelization.rbの修正です。

parallel testingでテストを実行する際にwith_info_handlerメソッドを使用するよう修正しています。

minitest-hooksのようなwith_info_handlerが実行される事に依存しているpluginを使用出来るようにする為。


Add collection cache versioning

activerecord/lib/active_record/relation.rbの修正です。

Relationでもrecyclable cache keyを使用出来るようActiveRecord::Relation#cache_versionメソッドを追加しています。

これにより、ActiveRecord::Relation#cache_keyが返すkeyにはrecordのcountやtimestampは含まれないようになっています。なお、この機能はデフォルトでは無効化されており、ActiveRecord::Base.collection_cache_versioningにtrueを指定するか、load_defaults6.0を指定した場合のみ使用されるようになっています。


Factorize bin/update in bin/setup, and make bin/setup idempotent

railtiesの修正です。

新規に作成するアプリケーションでbin/updateを生成しないよう修正しています。代わりに、updateの際もbin/setupを使用出来るよう、bin/setupでDBのセットアップをする際にdb:setupではなくdb:prepareを使用するよう修正しています。


Fix bin/update references after 8927eba83c

railtiesのテストの修正です。

Factorize bin/update in bin/setup, and make bin/setup idempotentで削除したbin/updateを参照している箇所があったのを削除しています。


Only retry tests on CI

railties/test/isolation/abstract_unit.rbの修正です。

CIでだけテストのリトライを行うよう修正しています。


Merge pull request #35994 from soartec-lab/update_guide_asset_pipeline_change_default

rails guideのThe Asset Pipelineの修正です。

デフォルトでは使用しなくなったcoffee-railsuglifier gemについての説明等が残っていたのを削除しています。


[ci skip] More asset guide touch ups

rails guideのThe Asset Pipelineの修正です。

長い行の適切な位置に改行を追加、言い回しの修正等を行っています。


Merge pull request #35984 from vishaltelangre/asn-misc-fixes

activesupport/lib/active_support/notifications/fanout.rbactivesupport/lib/active_support/notifications/instrumenter.rbの修正です。

Time.nowを使っていた箇所をConcurrent.monotonic_timeを使うよう修正、及び、integerを保持する為の変数の初期化にnilを使用していたのを0を使用するよう修正しています。


Prefer to indent and no blank line after private/protected [ci skip]

rails guideのContributing to Ruby on Railsの修正です。

Follow the Coding Conventionsの項に、private / protectedの後に空行は不要である旨説明を追加しています。


Merge pull request #35987 from kamipo/fix_dirty_tracking_after_rollback

Active Recordの修正です。

rollbackした場合に、dirty trackingで正しい値を取得出来ないバグがあったのを修正しています。これにより、rollbackした場合でも、attributesの変更前後の値が正しく取得出来るようになっています。


Add documentations to AS::Concern#included and #class_methods

activesupport/lib/active_support/concern.rbのdocの修正です。

ActiveSupport::Concern#included#class_methodsメソッドにdocを追加しています。


Removed unused require for active_support/deprecation

activesupport/lib/active_support/duration.rbの修正です。

使用していないactive_support/deprecationのrequireを削除しています。