なるようになるブログ

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

rails commit log流し読み(2017/07/23)

2017/07/23分のコミットです。

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


Merge pull request #29768 from janpieper/sort-enabled-extensions-in-dump

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

schema dumpにextensionsを出力する際に、sortした値を出力するよう修正しています。実行する度にdumpにdiffが出てしまう(sortしてないと、出力順が不定の為)のを避ける為。


Add task to verify a release.

Rakefiletasks/release.rbの修正です。

リリース作業で生成したgemファイルが問題無いか確認する為のverify taskを作成しています。

中身はrails newrails generate等のコマンドを実際に実行し、動作する事を確認するようになっています。


Support multiple versions in release announcement.

tasks/release.rbtasks/release_announcement_draft.erbの修正です。

複数バージョンのリリースアナウンスを同時に行えるよう、リリースアナウンス生成処理をリファクタしています。


Update release instructions in light of new tasks.

RELEASING_RAILS.mdの修正です。

先のコミットで修正した内容に合わせてReleasing Railsの説明を修正しています。


Remove dependency on from_now extension.

activesupport/lib/active_support/messages/metadata.rbの修正です。

Messages::Metadata#pick_expiryメソッドでexpireする時間を取得するのにfrom_nowメソッドを使用していたのを、使用しないよう修正しています。

-              expires_in.from_now.utc.iso8601(3)
+              Time.now.utc.advance(seconds: expires_in).iso8601(3)

Durationクラスへの依存を無くす為、ですかねえ。


Let Metadata instance handle purpose coercion.

activesupport/lib/active_support/messages/metadata.rbの修正です。

Messages::Metadata#initializeメソッドで引数のpurposeをStringに変換するよう修正しています。Metadataクラスの各クラスでpurposeを使用する際に、毎回Stringに変換するのを避ける為。


Merge pull request #29867 from koic/fix_a_failed_ar_test_when_using_oracle

activerecord/test/cases/relations_test.rbの修正です。

quoted_table_namequoted_primary_keyを取得するテストがOracle adapterでエラーになっていたのを修正しています。

-    assert_equal ["comments"], scope.references_values
+    if current_adapter?(:OracleAdapter)
+      assert_equal ["COMMENTS"], scope.references_values
+    else
+      assert_equal ["comments"], scope.references_values
+    end

Oracle adapterではquoted_table_namequoted_primary_keyの戻り値が大文字になっている為とのこと。