なるようになるブログ

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

rails commit log流し読み(2015/08/19)

2015/08/19分のコミットです。

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

guides/CHANGELOG.md


Adds a code of conduct

コントリビューターのcode of conductについて記載したCODE_OF_CONDUCT.mdを追加しています。

内容は、Contributor Covenant v1.2.0を記載しています。


drop string allocations for each resource

actionpack/lib/action_dispatch/journey/nodes/node.rbの修正です。

Symbolクラスで、nameの値を取得するのに、毎回メソッド呼び出し時に値を生成したいのを、コンストラクタで値を生成し、以降は生成した値を使いまわすよう修正しています。


symbols will always be constructed with strings. :bomb:

actionpack/test/journey/nodes/symbol_test.rbの修正です。

先の修正の影響で、Symbolクラスのインスタンスを生成する際、必ずstringを指定する必要があるようになったのですが、テストの中でnilを渡してしまっている箇所があったので、stringクラスの値を渡すよう修正しています。


drop array allocations when building paths

actionpack/lib/action_dispatch/journey/nodes/node.rbactionpack/lib/action_dispatch/routing/mapper.rbの修正です。

Mapping#build_pathでpathの生成処理を行う際、arrayの生成処理を減らせるよう、リファクタリングしています。

Symbolを取得する際、grepメソッドJourney::Nodes::Symbolクラスを集めていたのを、各NodeにSymbolかどうか判定用のメソッドを追加し、それを使用するよう修正する事で、配列を生成しないで済むよう修正しているようです。


don't touch internals

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

has_named_route?メソッドで、NamedRouteCollectionクラスの、内部情報であるroutes hashの値を直接参照していたのを、NamedRouteCollectionクラスのメソッド経由で参照するよう修正しています。


make the routes reader private

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

NamedRouteCollectionのroutes hashを可視性をprivateに変更しています。


Add code of conduct to README.md and to contributing guide [ci skip]

README.mdguides/source/contributing_to_ruby_on_rails.mdの修正です。

readmeとContributing to Ruby on Rails guideに、先に追加されたcode of conductへのリンクを追加しています。


Merge pull request #21282 from sjain1107/added_docs

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

TableDefinition#coloumnsメソッド、及びTableDefinition#remove_columnメソッドにdocを追加しています。