なるようになるブログ

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

rails commit log流し読み(2015/05/24)

2015/05/24分のコミットです。

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


add a branch to eliminate multiple nil checks

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

request idを取得する為のexternal_request_idメソッドの名前をmake_request_idメソッドに変更、及びrequest_idnilだった場合に、internal_request_idを返すよ修正しています。

external_request_idメソッドの戻り値を使用している箇所で、戻り値がnilだった場合にinternal_request_idを取得する、という処理を行っており、external_request_idメソッドに処理をまとめる事で、nilチェック処理を一つ減らしています。


remove useless ivar

actionpack/lib/action_dispatch/http/request.rbの修正です。

使用しなくなった@request_idを削除しています。


Reword a sentence about GH Issues in contributing guide.

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

https://github.com/rails/rails/issues へのリンクテキストをEveryone's Issuesからissues listに修正しています。 issueへのリンクテキストとして、Everyone's Issuesはちょっと違和感がある、という理由のようです。


Reword sentences in contributing docs for clarity.

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

guideの全体のグラマー/言い回しの修正を行っています。


use keyword arguments in HTTP request methods of assert_difference example [ci skip]

activesupport/lib/active_support/testing/assertions.rbのdocの修正です。

assert_differenceメソッドのexample内にあったhttp request用ヘルパーメソッドでキーワード引数を使用するよう修正しています。


Merge pull request #20260 from tgxworld/remove_deprecation_warning

actionview/test/actionpack/controller/render_test.rbの修正です。

deprecateになっているFile.exists?メソッドの代わりにFile.exist?メソッドを使用するよう修正しています。


Merge pull request #20265 from benpickles/gitignore-keeps

railties/lib/rails/generators/app_base.rbrailties/lib/rails/generators/rails/app/templates/gitignoreの修正です。

skip_keepsオプションを指定した場合、.gitignore.keepを記載しないよう修正しています。skip_keepsオプション指定した場合、そもそも.keepファイルを生成しないせず、不要な為、のようです。