なるようになるブログ

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

rails commit log流し読み(2017/08/04)

2017/08/04分のコミットです。

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

activesupport/CHANGELOG.md


Fix changelog format

activesupport/CHANGELOG.mdの修正です。

エントリー間の改行が不足していたのを追加しています。


Merge pull request #30039 from ricardotk002/fix-string-camelize-invalid-option

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

String#camelizeメソッドの引数に不正な値(:upper:lower以外の値)を指定した場合、ArgumentErrorをraiseするよう修正しています。


Add backticks around String#camelize

activesupport/CHANGELOG.mdの修正です。

`エントリー内のメソッド名にバッククォートを追加しています。


Merge pull request #30050 from kamipo/dont_pass_connection_to_statement_cache_execute

Active Recordの修正です。

StatementCache#initializeの引数にklassを追加しています。

元々はStatementCache#executeの引数にklassを渡すようになっていたのですが、実際は毎回同じklassを渡すようになっており、メソッド呼び出し時に毎回渡す必要は無い為初期化処理で渡すようにしています。合わせて、StatementCache#executeの引数からklassは削除しています。


Revise bundle binstub error message

railties/lib/rails/app_loader.rbの修正です。

Bundlerが生成したrailsコマンドを使用していた場合に出力エラーメッセージを改善しています。

元々はbinstubについての詳細な説明等が含まれていたのですが、不要そうな説明は削除し、更新するにはapp:update:binを実行すれば良い旨説明を修正しています。


Fix all rubocop violations

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


Remove unused LoadError::REGEXPS constant

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

使用していないLoadError::REGEXPS定数を削除しています。


Use Concurrent::Map than Mutex and Mutex_m for statement caches

activerecord/lib/active_record/core.rbactiverecord/lib/active_record/reflection.rbの修正です。

statement cacheの管理をmutext + Hashで行っていたのを、Concurrent::Mapを使用するよう修正しています。


Merge pull request #30022 from y-yagi/remove_extra_empty_line

railties/lib/rails/generators/rails/app/templates/Gemfileの修正です。

rails newで生成されるGemfilerubyのバージョン指定とgemの間に不要な空行があったのを削除しています。