なるようになるブログ

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

rails commit log流し読み(2018/12/03)

2018/12/03分のコミットです。

CHANGELOGへの追加はありませんでした。


Remove unnecessary variable route

actionpack/lib/action_dispatch/routing/mapper.rbの修正です。

make_routeメソッドから不要なroute変数を削除しています。


Remove obsolete yarn.lock files and check in root yarn.lock file

activestorage、actioncable配下にそれぞれyarn.lockがあったのを削除し、プロジェクトルートにyarn.lockを置くよう修正しています。

railsではYarn Workspacesを使用しており、Yarn Workspacesではプロジェクトルートにだけyarn.lockがあるのが想定している使用方法の為。


Merge pull request #34585 from marceloperini/marceloperini/33795

activestorage/test/service/s3_service_test.rbの修正です。

database/setupのrequireが不足していたのを追加しています。


Assigned but unused variable - bird

activerecord/test/cases/base_test.rbの修正です。

warning: assigned but unused variableのwarningが出ていたのを対応しています。


Add missing authorship to ActionCable changelog entry

actioncable/CHANGELOG.mdの修正です。

CHANGELOGのエントリーに対応者名の記載が漏れていたのを修正しています。


Address "warning: shadowing outer local variable - parts"

Active Recordの修正です。

warning: shadowing outer local variableのwarningが出ていたのを修正、及び、内部でだけ使用するREAD_QUERY定数の可視性をprivateに変更しています。


Fix NumericData.average test on ruby 2.6

activerecord/test/cases/calculations_test.rbの修正です。

BigDecimal gemがNilClassにto_dメソッドを追加するようになり、その影響でaverage`メソッドのテストがRuby 2.6でfailしてしまっていたのを修正しています。

元々はaveragenilを返す事を想定していたテストだったのですが、averageを返す際にto_dメソッドを使用している、かつ、NilClass#to_dBigDecimal(0)を返すようになった為failしていました。