なるようになるブログ

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

rails commit log流し読み(2016/10/20)

2016/10/20分のコミットです。

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


Fix indentation of code examples

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

ActiveRecord::ConnectionAdapters::Table#column_exists?#index_exists?メソッドのdoc内のコードexampleのインデントがずれていて、コードが正しく表示されてなかったのを修正しています。


Merge pull request #26829 from headius/wait_for_events_in_listen_tests

activesupport/test/file_update_checker_shared_tests.rbの修正です。

FileUpdateCheckerのテストで、ファイルの更新処理後にwait(sleep 1)をするよう修正しています。

JRubyでは、listenがファイルの更新イベントを非同期で取得する為、wait処理を入れないと更新を正しく検知出来ない為、との事です。


render_bind should be private

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

ActiveRecord::LogSubscriber#render_bindの可視性をprivateに変更しています。ActiveRecord::LogSubscriberクラスの中だけで使うメソッドだからですかねえ。


Merge pull request #26830 from headius/mask_forking_fsevent_test_on_jruby

activesupport/test/evented_file_update_checker_test.rbの修正です。

EventedFileUpdateCheckerのテストでforkしたprocessに対してnotify処理が正しく行われる事が確認するテストがあるのですが、JRubyではforkが無いので、該当のテストをJRubyではskipするよう修正しています。