なるようになるブログ

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

rails commit log流し読み(2017/09/19)

2017/09/19分のコミットです。

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

actionpack/CHANGELOG.md


Use the default Capybara registered puma server configuration

Action Packの修正です。

system testで使うようのPumaの設定をSystemTesting::Serverクラスで独自に行っていたのを、Capybaraに登録されている:puma serverの設定を使うよう修正しています。

Capybaraでの設定は下記の通り。

https://github.com/teamcapybara/capybara/blob/4fcc4a982254427a4578bc7ac582755c30e7bfe0/lib/capybara.rb#L449..L452

Rails側で独自に設定していたPumaの設定はではworker数を制限しておらず、config/puma.rbの設定内容によってはテストの際もCluster modeで動作していました。

ただ、テストの際はCluster modeで実行すべきではない(コネクション共有やリクエストのハンドリングの都合上)というのと、そもそもRails内部で独自に設定を持つ必要は無いのでは、という事でCapybaraに登録されている設定を使うようになりました。


Remove unused NumberToRoundedConverter#digits_and_rounded_number

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

使用していないNumberToRoundedConverter#digits_and_rounded_numberメソッドを削除しています。


Merge pull request #30648 from ydakuka/patch-1

rails guideの修正です。

guideのグラマー、リンク、exampleコードのフォーマットの修正等を行っています。