なるようになるブログ

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

rails commit log流し読み(2018/03/02)

2018/03/02分のコミットです。

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


ConsoleFormatter is no longer used as a class

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

ConsoleFormatterをclassからmoduleに変更しています。今はnamespaceとしてだけ使われており、classとしては使われていない為。


Use delegate private: true for SchemaCreation

activerecord/lib/active_record/connection_adapters/abstract/schema_creation.rbactiverecord/lib/active_record/connection_adapters/mysql/schema_creation.rbの修正です。

可視性をprivateにしたいメソッドをdelegateにするのに、delegateメソッドのprivate: trueオプションを使用するよう修正しています。


Don't expose verbose? helper method

activerecord/lib/active_record/tasks/database_tasks.rbの修正です。

内部でだけ使用するverbose?メソッドの可視性をprivateに変更しています。


Remove staled comment for JoinDependency#initialize

activerecord/lib/active_record/associations/join_dependency.rbのdocの修正です。

JoinDependency#initializeメソッドのdocを削除しています。

doc内に説明されている引数と実際の引数が異なっているのと、そもそもJoinDependencyクラスは内部用クラスな為。


PostgreSQL adapter also supports bulk alter since #31331 [ci skip]

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

change_tableメソッドのbulkオプションについて説明している箇所に、MySQL adapterでのみサポートしている旨説明が記載されていたのですが、Add bulk alter support for PostgreSQLPostgreSQL adapterでもサポートされたので、その旨説明を修正しています。


Ruby 2.4: take advantage of String#unpack1

unpackした値を取得するににunpack + first(または[0])を使用していたのを、String#unpack1メソッドを使用するよう修正しています。

参考:Feature #12752: Unpacking a value from a binary requires additional '.first'


Remove unnecessary respond_to?(:report_on_exception) checking

各テストでThread.report_on_exceptionの値を変更する前にreport_on_exceptionが定義されているかチェックしていたのを、チェックしないよう修正しています。masterはRuby 2.4以上をサポートする為、必ずreport_on_exceptionは定義されている為。