なるようになるブログ

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

rails commit log流し読み(2015/09/21)

2015/09/21分のコミットです。

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

activesupport/CHANGELOG.md


:scissors: empty line at the top of files

幾つかのファイルにの先頭に不要な空行があったのをまとめて削除しています。


Typppo

actionpack/lib/action_dispatch/middleware/exception_wrapper.rbの修正です。

applicationappplicationにタイポしている箇所があったのを修正しています。


:hocho: Typos

幾つかのテストファイルのメソッド名、変数にタイポがあったのをまとめて修正しています。


Add title for key lengths for multiple keys.

activerecord/lib/active_record/connection_adapters/abstract/schema_statements.rbのdocの修正です。

add_indexメソッドのdoc内、複数キーでindexを作成する場合のexampleについて説明記載している箇所にタイトルが無かったのを追加しています。


Update routing.rb

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

Routing moduleのdoc内のexampleにendが足りてない箇所があったので、追加しています。


Applying right result of examples in ActiveSupport Multibyte [ci skip]

activesupport/lib/active_support/core_ext/string/multibyte.rbのdocの修正です。

String#mb_charsメソッドのdoc内のexampleの実行結果が実際の結果と異なっていたのを修正しています。


Short-circuit blank? on date and time values

Active Supportの修正です。

date / time関係の各クラスにblank?メソッドを追加しています。

class Date #:nodoc:
  # No Date is blank:
  #
  #   Date.today.blank? # => false
  #
  # @return [false]
  def blank?
    false
  end
end

同じ内容のblank?メソッドDateTimeTimeTimeWithZoneクラスに追加されています。

date / time クラスの値がblankになる事は無いのですが、 明示的にメソッドを定義する事で、親クラスまで処理がいかなくなり、少し高速化する為、明示的にメソッドを定義したとの事です。


Use master version of Turbolinks gem

Gemfileの修正です。

masterのturbolinksを使用するよう修正しています。リリース済みのturbolinksだと、alias_method_chainを使用しており、deprecateメッセージが表示されてしまう為、との事です。


Silence logging in Active Job unit tests

activejob/test/helper.rbの修正です。

Active Jobのユニットテスト実行時にログが出力されないよう、Loggerにnilを設定しています。


Run rake db:migrate for all Active Job integration tests

activejob/test/support/integration/dummy_app_template.rbの修正です。

Active Jobのインテグレーションテストで、必ずdb:migrate を実行するよう修正しています。

テストではマイグレーションファイルは不要なのですが、db:migrateを実行しないとワーニングが出力されてしまうため、ワーニングが表示されるのを避ける為に必ず実行するようにしたとの事です。


Use Sidekiq.options to set initial wait

activejob/test/support/integration/adapters/sidekiq.rbの修正です。

Sidekiq.options[:poll_interval_average]に1を設定しています。

poll_interval_averageはinitial waitを設定する為のパラメータで、デフォルトでは10〜15秒の値がランダムで設定されるのですが、ランダムに値が設定される事によりSidekiqを使用したActive Jobのテストが時折コケてしまっていたので、小さな数字を明示的に設定するようにしています。


Update to latest turbolinks

Gemfile.lockの修正です。

turbolinksのバージョンを最新化しています。


Merge pull request #21612 from ronakjangir47/remove_unused_params

actionview/lib/action_view/lookup_context.rbの修正です。

LookupContext.register_detailメソッドの引数から、使用していないoptions引数を削除しています。