なるようになるブログ

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

rails commit log流し読み(2019/12/28)

2019/12/28分のコミットです。

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


Merge pull request #38086 from yhirano55/activerecord/allow_enum_definitions_with_boolean_values

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

enumの定義にbooleanの値を使用出来るよう修正しています。


Switch to standardized argument name

actionpack/lib/abstract_controller/helpers.rbの修正です。

helper_methodメソッドの引数の変数名をmethsからmethodsに修正しています。j


Call method outside the loop

actionview/test/template/form_helper/form_with_test.rbの修正です。

同じメソッドに対するruby2_keywordsの呼び出しが複数回行われていたのを、一度だけ行うよう修正しています。


Fix test with new capybara

Gemfile.lockactionpack/test/dispatch/system_testing/driver_test.rbの修正です。

各gemを最新バージョンに更新、及び、system testのテストが最新のCapybaraで通らなくなっていた(browser_optionsの戻り値が変わっていた)のを対応しています。


Downgrade parser to work correctly with Ruby 2.5

Gemfile.lockの修正です。

先のgemの更新であげたparser gemを2.6系に戻しています。最新のバージョンだとRuby 2.5で動作しない為。


remove reference to global rails command and replace with bin/rails

各ドキュメント、コードでrailsのコマンドを実行するのにrailsを使うようになっていた箇所をまとめてbin/railsを使うよう修正しています。

Merge pull request #33229 from albertoalmagro/albertoalmagro/prefer-rails-command-over-bin-railsrailsを使うよう修正されたのですが、グローバルのコマンドに依存すると予期せぬ挙動になる事がある為、bin/railsを使うよう戻しています。


Remove warning from the deprecation method wrappers

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

MethodWrapperでキーワード引数に関するwarningが表示されないよう対応しています。


No need to split the keyword arguments here

activesupport/lib/active_support/core_ext/object/try.rbの修正です。

trytry!メソッドの引数から使用していないキーワード引数の指定を削除しています。


Split keyword argument in the geenrators action tests

railties/test/generators/actions_test.rbの修正です。

actionメソッドでキーワード引数に関するwarningが表示されないよう対応しています。

が、Ruby 2.6以下で動作しない対応内容になっていた為、後ほど再度修正しています。


Deal with Ruby 2.5 and Ruby 2.6 in a different way

railties/test/generators/actions_test.rbの修正です。

actionメソッドがRuby 2.6以下でも動作するようRubyのバージョンによって引数の扱いを変えるよう修正しています。


Fix rubocop violations

railties/test/generators/actions_test.rbの修正です。

rubocopの設定に違反している箇所があったのを修正しています


Address DEPRECATED: use MT_CPU instead of N for parallel test runs

actionpack/test/abstract_unit.rbの修正です。

Action Packのテストでprocess数を指定するための環境変数名にNを使用していたのを、MT_CPUを使用するよう修正しています。

このNはminitest由来なのですが、minitest側が5.12.0でNを使用するのをdeprecateにして代わりにMT_CPUを使用するのを推奨しており、それと合わせる為。


Remove passing a block which is unneeded

actionpack/lib/action_controller/metal.rbactionpack/lib/action_dispatch/middleware/stack.rbの修正です。

MiddlewareStack::Middleware classの各メソッドの引数からbuild_blockを削除しています。

ruby2_keywordsで指定したメソッドの引数はdelegateしてもマークがそのままになっており、別途blockを指定して渡す必要がなかった為削除しています。