なるようになるブログ

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

rails commit log流し読み(2018/08/20)

2018/08/20分のコミットです。

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

railties/CHANGELOG.md


DRY in Active Record Query Interface [ci skip]

rails gudieのActive Record Query Interfaceの修正です。

[Explained difference between scope & class method](https://github.com/rails/rails/pull/33653)で追加された、scopeとclass methodの違いについての説明を削除しています。

Using conditionalsの項にまったく同様の説明が既にある為。


activestorage: yarn build

activestorage/app/assets/javascripts/activestorage.jsの修正です。

yarn buildを実行してactivestorage.jsを再生性しています。

Support multiple submit buttons in Active Storage formsの際に、手でファイルを編集してしまったようで、buildをすると差分が出てしまうようになった為。


Improve test case to test enum correctly

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

association + enumのテストで、テスト用のmodelにenumの定義が不足していたのを修正しています。


SQLite3: Fix rename reference column not to lose foreign key constraint

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

SQLite3 adapterで、schemaでreference columnをリネームした際に、foreign keyの情報が失われてしまうバグがあったのを修正しています。


Merge pull request #33631 from anniecodes/initializers-command

railties/lib/rails/commands/initializers/initializers_command.rbrailties/lib/rails/tasks/initializers.rakeの修正です

rake taskとして実装されていたinitializersrails commandとして実装しています。

これにより、rakeでの実行(rake initializers)はdeprecateになっています。


Don't put up with not saying it.

railties/lib/rails/commands/initializers/initializers_command.rbの修正です。

結果を表示するのにputsメソッドを使用していたのを、他のコマンドと合わせてsayコマンドを使用するよう修正しています。


Merge pull request #33621 from sikachu/sikachu-guide-rubocop

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

Follow the Coding Conventionsの項に、RuboCopを使っている旨説明、及び、PR投げる前にRuboCopをローカルで実行してね、という説明を追加しています。


Merge pull request #31132 from emaxi/feature/add-missing-documentation-option-to-number-to-currency

actionview/lib/action_view/helpers/number_helper.rbactivesupport/lib/active_support/number_helper.rbのdocの修正です。

number_to_currencyメソッドのdocにstrip_insignificant_zerosオプションについての説明を追加しています。


Fix rails routes -c for controller name consists of multiple word.

rails routes-c(controllerオプション)を指定、かつ、controller名に複数の単語が含まれているcontroller名を指定した場合に、routesが正しく表示されないバグがあったのを修正しています。


Merge pull request #33652 from tgxworld/improve_tests_pool_config

activerecord/test/cases/connection_pool_test.rbactiverecord/test/cases/reaper_test.rbの修正です。

ActiveRecord::ConnectionAdapters::ConnectionPoolのテストの改善として、idle_timeout configを指定した場合等の不足していたテストを追加、timeoutを明示的に指定してテスト実行時間が短くなるよう修正、等を行っています。